-
Notifications
You must be signed in to change notification settings - Fork 160
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
Proxy-Authorization added #59
base: master
Are you sure you want to change the base?
Conversation
Thanks, confirm it is working! |
Thanks for this, can confirm its a big issue, this is forcing me to use forks rather than the main package |
Do not install typing on python 3.7+
@StasDeep, is it possible to merge and close this PR. Seems like it's an actual problem for some engineers. |
@dtalkachou I don't have merge permissions on this repository. |
Unfortunately I lost permissions myself :( We should probably fork this repo to another place, likely scrapy-plugins. |
@kmike, Is there anybody else who has permissions? |
|
Seems like I didn't have it enabled. Yes, after code reading I think that |
I think it's an important feature because not always people have an unauthorized proxy.
We had the following error before:
Please, update PyPI after merging this request.