-
Notifications
You must be signed in to change notification settings - Fork 3
feat: add TransportOptions for configuring TLS, proxy, and default headers #122
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
Open
mridang
wants to merge
26
commits into
beta
Choose a base branch
from
feat/add-transport-options
base: beta
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
887d007
Add TransportOptions for configuring TLS, proxy, and default headers
mridang 0444f7f
Fix hostname verification for custom CA and address review feedback
mridang d3fcc5c
Merge custom CA with default trust store instead of replacing it
mridang 31bccb0
Fix SSL socket resource leak in transport options test
mridang e7def42
Apply transport options to OAuth token exchange requests
mridang 2f29aef
Fix custom CA cert test and apply transport options to token exchange
mridang c34990c
Standardize transport options tests across SDKs
mridang 2bb9aa1
Add null guard for transportOptions in ApiClient constructor
mridang 55beb47
Add null guards for transportOptions in all public constructors
mridang 1dd95e3
Validate header name and value in TransportOptions builder
mridang 16c7a55
Centralize SSL context building in TransportOptions
mridang 8ee3d24
Verify default headers on API calls via WireMock verification
mridang 728e978
Fix SpotBugs warning and proxy test reliability
mridang 39765d7
Cache SSLContext in TransportOptions to avoid per-request rebuilds
mridang ef23bed
Fix install command from composer to Maven dependency
mridang ee976f4
fix: remove unnecessary SpotBugs suppression from buildSSLContext
mridang 6677286
build: enable unused import removal in Spotless
mridang 604bbd3
build: reorder Spotless steps to remove unused imports before formatting
mridang 06365f3
fix: remove unused imports from ApiClient and OAuthAuthenticator
mridang 6345a39
Add real proxy container to transport options test
mridang 3d9cff2
chore: align docs and remove inline comments
mridang c9f4747
replace tinyproxy with ubuntu/squid:6.10-24.10_beta
mridang 17bc888
catch GeneralSecurityException explicitly instead of broad Exception
mridang 60db143
docs: fix proxy auth docs to use URL credentials instead of default h…
mridang bb84212
fix: replace broad catch(Exception) with specific checked exceptions
mridang fb43253
fix: remove stale SpotBugs filters for moved trust manager and except…
mridang File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
This file contains hidden or 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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.