#153 Remove fedora cert
Merged by ausil. Opened by cqi.
cqi/fedpkg remove-fedora-cert  into  master

Download 153.patch

Kerberos authentication is used by dist-git and lookaside already.
Remove cert related code.

Signed-off-by: Chenxiong Qi cqi@redhat.com

Please add an option to write out a .fedoa.upn file

The removal looks good to me.

@ausil I just searched .fedora.upn here https://pagure.io/fedora-packager/c/715483c1bbdf5cceaeb63e90410139cd2f02abe7?branch=master. fedora-cert is able to write out such a file. I have no idea why fedpkg should also be able to do this.

It's used in some places to set your fedora user name. It would allow us to remove fedora-cert entirely. Once everything depending on it removed the use of fedora-cert.

@ausil Sorry, I'm still confused. Could you please file an issue with a use case of fedpkg and .fedora.upn? Maybe this could be fixed in another PR.

The removal here breaks the usage of the .fedora.upn file

rebased onto 67c5a9058a358ed4fc3bdf8842841260d5a06520

@ausil PTAL

@ausil PTAL

I have no idea what that means

Pull-Request has been merged by ausil

Metadata