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

proxy authentication for master #216

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

key2peace
Copy link
Contributor

tested with 3cx

@key2peace key2peace changed the title prroxy authentication for master proxy authentication for master Jan 15, 2024
@ollien
Copy link

ollien commented Apr 9, 2024

FWIW, I have a private fork of this library which uses this patch, and I had great success with it on a server that required Proxy-Authentication. Would be cool to see it in the stable branch :)

@avocadio
Copy link

avocadio commented May 5, 2024

@tayler6000 I'm mindful of all the work that is being done in the v2 and likely release, but would you mind reviewing this PR - it works for me as well.

@the-master-student
Copy link

Hi everyone,

I am new to the topic and feel a bit helpless. I have checkout these changes, however I still get a proxy authorization error. How would you register and initiate a call with those changes you did on this branch?

Thank you in advance and grateful for any help.

@vinod-sarvam
Copy link

Is this PR pending some review or has some challenges preventing it from being merged to the master? Can the author or someone please comment on the same?

@toninog
Copy link

toninog commented Nov 24, 2024

FWIW, I have a private fork of this library which uses this patch, and I had great success with it on a server that required Proxy-Authentication. Would be cool to see it in the stable branch :)

you able to make this public?

@ollien
Copy link

ollien commented Nov 24, 2024

FWIW, I have a private fork of this library which uses this patch, and I had great success with it on a server that required Proxy-Authentication. Would be cool to see it in the stable branch :)

you able to make this public?

No, sorry. I no longer work where I had access to this. That said, it wasn't anything special beyond cherry-picking this PR in, iirc

@YaPlusDePile
Copy link

Hi,
I went to try this patch with my 3CX server. It seem to have a 407 Proxy Authentication Required on trying to make a call. did you have tryed this ?

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.

7 participants