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.

Sign up for GitHub

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

Jump to bottom

Uncaught exception: AssertionError [ERR_ASSERTION] #51

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

Uncaught exception: AssertionError [ERR_ASSERTION] #51

wants to merge 1 commit into from

Conversation

pavelbinar
Copy link

#43

@pavelbinar pavelbinar mentioned this pull request Jan 1, 2021
Open
@benjaminvanrenterghem
Copy link

Please merge this pull request. It's causing unnecessary problems for people using HTTP proxies since 2016.

#43
#20

image

@pavelbinar
Copy link
Author

👏🏻

@9178angel
Copy link

hey pavelbinar can we merge this PR? pls :D

@pavelbinar
Copy link
Author

I am not the maintainer of this repository, so I cannot do that.
I guess someone from this guy has to do that: https://github.com/orgs/request/people

@9178angel
Copy link

@mikeal can we merge this PR? :D

@pavelbinar
Copy link
Author

@9178angel you can install the fixed version from the repository directly:

https://github.com/remoteorigin/tunnel-agent

https://stackoverflow.com/questions/17509669/how-to-install-an-npm-package-from-github-directly

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@IvanMMM IvanMMM IvanMMM approved these changes

@9178angel 9178angel 9178angel approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
4 participants
@pavelbinar @benjaminvanrenterghem @9178angel @IvanMMM