koji

#2042 add-host work even if host already tried to log in
Merged by tkopecek. Opened by tkopecek.
tkopecek/koji issue1874  into  master

Download 2042.patch

Fixes: https://pagure.io/koji/issue/1874

Metadata Update from @tkopecek:
- Pull-request tagged with: testing-ready

pretty please pagure-ci rebuild

rebased onto f1df5c298809b3444a15c6330d2109cfc0dd0d7c

pretty please pagure-ci rebuild

Metadata Update from @tkopecek:
- Pull-request untagged with: testing-ready

rebased onto b89e1895108da1bc616a3a815aa7a071f1cb1624

@mikem I've changed the logic. Let's assume, that NORMAL user was already created and let admin --force it to convert it to HOST. Does it make sense?

rebased onto 2cfa19af24c5c75e24b12ac120be291871333641

the key in "userInfo" should be krb_principal
And is this userInfo too strict? not sure if we really care about the username

1) Fixed
2) I would be more cautious here. So, checking also the username seems ok to me. But maybe we can be more relaxed.

1 new commit added

  • fix get_user params

ok, :thumbsup:

Metadata Update from @julian8628:
- Pull-request tagged with: testing-ready

1 new commit added

  • fix test

1 new commit added

  • query on krb only if it is requested

1 new commit added

  • check krb_principal before it is rewritten

Metadata Update from @jcupova:
- Pull-request tagged with: testing-done

1 new commit added

  • fix tests

Commit 934b3c95 fixes this pull-request

Pull-Request has been merged by tkopecek
Metadata
None

testing-done testing-ready

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.