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 support to proxy servers #102

Open
Godd67 opened this issue Sep 1, 2017 · 7 comments
Open

Add support to proxy servers #102

Godd67 opened this issue Sep 1, 2017 · 7 comments
Labels
enhancement Introduces a new feature or improvement proposal Proposal or draft for a major change triage Issue is being researched

Comments

@Godd67
Copy link

Godd67 commented Sep 1, 2017

We have a few customers that do not have direct Internet connection from their shipping machines because of company polices and instead are using local network proxy servers to communicate with outside world. For instance a customer’s PC can’t access https://onlinetools.ups.com/webservices (UPS API endpoint) directly , instead we let them to enter their proxy server URL and credentials in our software and set underlying connection’s (either SoapHttpClientProtocol or HttpWebRequest) Proxy property. If it is set, all outbound requests goes through this proxy URL. We would like to see similar proxy support in Easypost client.

Thanks.

@att14
Copy link
Contributor

att14 commented Sep 1, 2017

You can change the underlying url to make requests to by setting it on ClientConfiguration.

https://github.com/EasyPost/easypost-csharp#configuration

Does this solve your issues?

@Godd67
Copy link
Author

Godd67 commented Sep 1, 2017

In our case proxy server needs additional credentials that we need to pass. System.Net.Webproxy handles it .

@att14
Copy link
Contributor

att14 commented Sep 1, 2017

What kind of authentication are you using? Basic Auth?

@Godd67
Copy link
Author

Godd67 commented Sep 1, 2017

Honestly, not sure. We are setting this property - https://msdn.microsoft.com/en-us/library/system.net.webproxy.credentials(v=vs.110).aspx

@att14
Copy link
Contributor

att14 commented Sep 1, 2017

This could be a pretty major change to how the client and request work. I have been wanting to make a change that would support something like this. I'll take a look and see how complicated it would be.

@att14 att14 self-assigned this Sep 1, 2017
@att14 att14 removed their assignment Jan 13, 2021
@Justintime50
Copy link
Member

This items isn't on our immediate roadmap; however, we are open to pull requests if someone wants to take a stab at it.

@nwithan8 nwithan8 added the proposal Proposal or draft for a major change label Mar 29, 2022
@nwithan8
Copy link
Member

@Godd67 I am exploring the scope of the work for this. It seems fairly trivial. Is this still a requirement?

@nwithan8 nwithan8 added the triage Issue is being researched label Aug 17, 2022
@nwithan8 nwithan8 added the enhancement Introduces a new feature or improvement label Feb 17, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Introduces a new feature or improvement proposal Proposal or draft for a major change triage Issue is being researched
Projects
None yet
Development

No branches or pull requests

4 participants