Add HTTP proxy support #56
Merged
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
Motivation: Adding HTTP proxy support to allow localproxy to used TCP tunneling to connect using TCP tunneling. Modifications: 1. Refactored the tcp_proxy_config into a separate header file and renamed it to LocalproxyConfig to make it easier to use with the new component. And added some field descriptions. 2. Added a field for the tcp_adapter_proxy class for the HttpsProxyAdapter. 3. Separated out the TCP socket definition from the websocket stream definition to make it accessible. 4. Added implementation for HttpsProxyAdapter class. 5. Added integration for HttpsProxyAdapter errors and boost errors 6. Added code that starts an HTTP server that I can use to test against 7. Added unit test cases for the public methods in the HttpsProxyAdapter class 8. Added a wrapper around Boost websocket stream so that the caller won't have to worry about the underlying websocket stream type which will vary based on whether the web proxy connection is over TLS or not. 9. Updated the HttpsProxyAdapter to perform the SSL handshake with the web proxy before sending HTTP CONNECT when needed. 10. Updated the main function to accept https_proxy URLs with HTTPS protocol. 11. Adjusted the unit tests.
MiaoZhangAWS
approved these changes
Jul 13, 2021
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.
Motivation
Modifications
Change summary
component. And added some field descriptions.
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.