-
-
Notifications
You must be signed in to change notification settings - Fork 798
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
Support another HTTP Proxy methods #299
Support another HTTP Proxy methods #299
Conversation
Thank you for your PR, please consider #304 |
Hi @ittus , thank you for your PR! I'll be merging it another time after I review it more thoroughly. Thank you for your patience also <3 |
Could you please validate this pull request ? I applied these changes and it works for me. |
…t#299 but not applied in main repo, so had to clone it
My team is also interested in this fix. Could someone take a look at it. Can confirm it works. |
The code looks good to me. However, many changes have been introduced since 2017. Could you rebase the master branch and test if it stills works? |
@ansraliant I've rebased and checked on my personal project. Everything is working as expected. |
It looks good to me. I think one of the maintainers can merge it. |
Fix #298