PR#2242: python: return the correct message when all request attempts fail - - Pagure.io

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

copr / copr

#2242 python: return the correct message when all request attempts fail

Merged Jul 18, 2022 by frostyx. Opened Jul 12, 2022 by frostyx.

copr/ frostyx/copr invalid-login into main

Download 2242.patch

frostyx commented Jul 12, 2022

Fix #2235

zuul commented Jul 12, 2022

Build succeeded.

csdiff-pylint : SUCCESS in 1m 37s

praiskup commented Jul 13, 2022

Do we want to have this in two places (send, and the inner _send_request_repeatedly()?

1 new commit added

python: handle request errors only once

Jul 18, 2022

frostyx commented Jul 18, 2022

You are right, it is enough to have just one handle_errors call. PTAL.
If you like this change better, I will squash the commits.

zuul commented Jul 18, 2022

Build succeeded.

csdiff-pylint : SUCCESS in 1m 04s

rebased onto 32b7c26af63b6ec023ec9f8eca82078ab037c33f

Jul 18, 2022

zuul commented Jul 18, 2022

Build succeeded.

csdiff-pylint : SUCCESS in 1m 05s

rebased onto 5572bff73596f0e96534bbd668f0dc35a9b077d3

Jul 18, 2022

zuul commented Jul 18, 2022

Build succeeded.

csdiff-pylint : SUCCESS in 1m 01s

Pull-Request has been merged by frostyx

Jul 18, 2022

Metadata

Assignee

None

Tags

No Tags

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.