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

More debug logging #139

Merged
merged 8 commits into from Nov 23, 2023
Merged

More debug logging #139

merged 8 commits into from Nov 23, 2023

Conversation

edsu
Copy link
Contributor

@edsu edsu commented Nov 1, 2023

To help diagnose when things are going wrong this commit adds some additional debug logging when requests are happening, what parameters are being used, sleep behavior and error responses from web.archive.org.

Closes #138

To help diagnose when things are going wrong this commit adds some
additional debug logging when requests are happening, what parameters
are being used, sleep behavior and error responses from web.archive.org.

Closes edgi-govdata-archiving#138
Copy link
Member

@Mr0grog Mr0grog left a comment

Choose a reason for hiding this comment

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

Some minor nits on the messaging, but this looks good. Thanks for flagging and adding a PR for it!

wayback/_client.py Outdated Show resolved Hide resolved
wayback/_client.py Outdated Show resolved Hide resolved
wayback/_client.py Outdated Show resolved Hide resolved
wayback/_client.py Outdated Show resolved Hide resolved
wayback/_utils.py Outdated Show resolved Hide resolved
@Mr0grog Mr0grog merged commit 6baec33 into edgi-govdata-archiving:main Nov 23, 2023
17 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

More logging
2 participants