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

Add a generic UA for sites that are being picky #1666

Merged
merged 1 commit into from
Aug 23, 2019

Conversation

Martii
Copy link
Member

@Martii Martii commented Aug 23, 2019

NOTE:

  • This could potentially still throw ECONNRESET if that server blocks request. This is called browser sniffing and usually isn't a wise thing to do. Not sure if all nginx servers do this.

Applies to https://openuserjs.org/garage/Cant_add_my_script


Loosely applies to historical #1332

NOTE:
* This could potentially still throw ECONNRESET if that server blocks `request`. This is called browser sniffing and usually isn't a wise thing to do. Not sure if all nginx servers do this.

Applies to https://openuserjs.org/garage/Cant_add_my_script
@Martii Martii added CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. remote issue Sometimes used for remote issues. Could be S3, could be mLab, etc. labels Aug 23, 2019
@Martii Martii merged commit db4e646 into OpenUserJS:master Aug 23, 2019
@Martii Martii deleted the addUA branch August 23, 2019 17:35
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Apr 30, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CODE Some other Code related issue and it should clearly describe what it is affecting in a comment. remote issue Sometimes used for remote issues. Could be S3, could be mLab, etc.
Development

Successfully merging this pull request may close these issues.

1 participant