Default Proxy Causes failures if other proxy is not used #30
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Ref #17 |
|
Yes and we have this issue logged and will provide solution shortly |
any update on this one? It has been nearly 2 years. I provided a valid change that solves this issue and has been proven to be working in our production environment for that time period... |
|
Bump @amie-gao this PR is still open. As a CC processor i'd expect that many of your impls are in hardened environments that may need proxy access for outbound requests but not for local traffic. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Opening the soap connection and setting the default proxy leaves the rest of the asp.net app in a default proxy state which may not be intended, the specific connection should be using the proxy and not app wide.
This also covers the proxy being anon access and not just for a user cred