PR#55: Fix inverting the user cache - fedora-infra/distgit-bugzilla-sync

fedora-infra / distgit-bugzilla-sync

#55 Fix inverting the user cache

Merged 3 years ago by nphilipp. Opened 3 years ago by pingou.

fedora-infra/ pingou/distgit-bugzilla-sync master into master

Fix inverting the user cache

Pierre-Yves Chibon • 3 years ago

438b5c8

distgit_bugzilla_sync/script.py

file modified

+1 -1

		`@@ -228,7 +228,7 @@`
		`that the bugzilla_email is the key and the username the value.`
		`"""`
		`for username in self.user_cache:`
		`- bz_email = self.user_cache[username]["bugzilla_email"].lower()`
		`+ bz_email = self.user_cache[username]["email"].lower()`
		`self.inverted_user_cache[bz_email] = username`

		`def _get_bugzilla_email(self, username):`

pingou commented 3 years ago

In the previous commit we changed the key from bugzilla_email
to simply email to be consistent with the data we retrieve
from FAS, but it looks like we forgot to update this method which
thus results in the script failing with a KeyError on
bugzilla_email.

Signed-off-by: Pierre-Yves Chibon pingou@pingoured.fr