pagure

#2311 Fix KeyError in logcom_server and loadjson_server
Merged by mprahl. Opened by mprahl.
Unknown source master

Download 2311.patch

@puiterwijk found this bug in "pagure_logcom_server.py". I saw the same issue in "pagure_loadjson_server.py" but I don't know if it applies for sure.

I recommend adding tests for this change to ensure that it stays fixed.

I currently don't have the cycles to add tests for this, but as long as it functionally works for you guys, I suggest we merge it as is since it is a bug fix that's needed.

Going to merge this one manually, thanks for the patch!

There are no tests for this yet so that would/should be a PR on its own.

Commit d7d7893f fixes this pull-request

Pull-Request has been merged by mprahl@redhat.com
Metadata
None
No Tags
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.