ipsilon

#100 Add Fedora SSH and GPG key mapping and make sure OpenID AX works with multiple values
Merged by puiterwijk. Opened by puiterwijk.
puiterwijk/ipsilon add-ssh-key  into  master

Download 100.patch
no initial comment

Should we check if val has actually a value? (ie is not an empty line?)

2 new commits added

  • Split multiple OpenID AX values
  • Add ssh and gpg key id mappings to authfas

Looks good to me :)

rebased

is it safe to assume val has been stripped?

one question, otherwise looks sane to me

Well, if it's an empty string, that's just fine problem. The only problem is we need to be careful sending empty strings.

If it's a string with just whitespace, that's just fine. is what I meant to say.

Ok, but does it make sense?

rebased

:thumbsup: for me

rebased

Pull-Request has been merged by puiterwijk
Metadata
None
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.