#155 Fix for rhbz #1412260
Merged 5 years ago by kevin. Opened 6 years ago by sergiomb.
Unknown source master  into  master

@@ -29,8 +29,6 @@

      Read in the certificate so we dont duplicate the code 

      """

       # Make sure we can even read the thing.

-     if os.path.exists(os.path.expanduser('~/.fedora.upn')):

-         return None

      cert_file = os.path.join(os.path.expanduser('~'), ".fedora.cert")

      if not os.access(cert_file, os.R_OK):

          raise fedora_cert_error("""!!!    cannot read your ~/.fedora.cert file   !!!

Return None in _open_cert when ~/.fedora.upn exist is not correct,
i.e. we can read cert if we delete ~/.fedora.upn ?
With latest commits we already have enough warnings that fedora does
not use client certificate anymore, but if other external repos try to use this
code, the code works instead have an traceback error.

Signed-off-by: Sérgio M. Basto sergio@serjux.com

:thumbsup: for me, works fine applying this patch on Fedora 27

@sergiomb is this still needed? Can you rebase it?

rebased onto c26bc96

5 years ago

Excellent, thanks for the PR!

Pull-Request has been merged by kevin

5 years ago
Metadata