Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Give a reasonable error for unreachable wiki url #303

Merged
merged 1 commit into from
Jul 11, 2023
Merged

Conversation

psss
Copy link
Owner

@psss psss commented Mar 31, 2023

Prevent traceback when wiki server is not reachable. Fix BZ#2118329.

@psss psss requested review from kwk and lukaszachy March 31, 2023 15:29
@psss psss added this to the 0.21 milestone Mar 31, 2023
@psss psss force-pushed the unreachable-wiki branch 2 times, most recently from 9cfc82c to 3d45c37 Compare June 30, 2023 07:59
Copy link
Collaborator

@lukaszachy lukaszachy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about catching also xmlrpc errors?
This patch works for 'unreachable' however when I used example config ..

 ERROR  <Fault 1: 'This moin wiki does not allow xmlrpc method calls.'>

* Updates on wiki test: 0 changes of 0 pages

IMO it should handle that as well

@psss
Copy link
Owner Author

psss commented Jun 30, 2023

Yeap, makes sense, fixed in 67cffa0.

@psss psss self-assigned this Jun 30, 2023
@psss psss requested a review from lukaszachy June 30, 2023 21:06
Prevent traceback when wiki server is not reachable.
Fix BZ#2118329.
@psss psss merged commit 101d208 into main Jul 11, 2023
12 of 13 checks passed
@psss psss deleted the unreachable-wiki branch July 11, 2023 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants