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

Specify language in scraper so it doesn't look for the wrong thing #192

Closed
wants to merge 1 commit into from

Conversation

mrsrec
Copy link
Contributor

@mrsrec mrsrec commented Nov 13, 2023

Depending on a web server's configuration or location, it may get back a response with a language besides English, breaking the scraper. This fixes that.

This does not currently affect en.scratch-wiki.info because it doesn't put an Accept-Language header by default, and Scratch ignores IP addresses, but it will affect other web servers and may affect en.scratch-wiki.info as well should either of the two aforementioned things change.

@jacob-g
Copy link
Member

jacob-g commented Nov 13, 2023

While I don't think this is likely to be a problem, I suppose it doesn't hurt to be safe. I'll validate this when I get the chance.

@mrsrec mrsrec closed this by deleting the head repository Dec 6, 2023
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.

2 participants