PR#155: Fix for rhbz #1412260 - fedora-packager

fedora-packager

#155 Fix for rhbz #1412260

Merged 6 years ago by kevin. Opened 7 years ago by sergiomb.

Unknown source master into master

Fix for rhbz #1412260

Sérgio M. Basto • 6 years ago

c26bc96

src/fedora_cert/__init__.py

file modified

-2

		`@@ -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 !!!`

sergiomb commented 7 years ago

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