-
Notifications
You must be signed in to change notification settings - Fork 2
feat: add TransportOptions for configuring TLS, proxy, and default headers #79
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
Merged
+592
−70
Merged
Changes from all commits
Commits
Show all changes
42 commits
Select commit
Hold shift + click to select a range
b8b69fd
Add TransportOptions for configuring TLS, proxy, and default headers
mridang 4e456f4
Fix hostname verification for custom CA certificates
mridang d591eff
Merge custom CA with default trust store instead of replacing it
mridang 3f807ba
Fix contradictory README example showing insecure with ca_cert_path
mridang 316b199
Fix SSL socket resource leak in transport options test
mridang 666391b
Apply transport options to OAuth token exchange requests
mridang ab4a73e
Fix custom CA cert test and apply transport options to token exchange
mridang 111c861
Standardize transport options tests across SDKs
mridang c8d3c36
Copy frozen default_headers before assigning to mutable config
mridang 7c8e778
Extract config mutation helper to eliminate duplication across factor…
mridang 15ee731
Centralize connection opts building in TransportOptions
mridang 0a03429
Verify default headers on API calls via WireMock verification
mridang 2d71b76
Fix README: correct HTTP library name and debug example
mridang 12e91ee
Fix rubocop offenses in transport options and tests
mridang 87f8dbc
Remove individual transport params from factory methods
mridang 9ac8dd2
Use withAccessToken for proxy test reliability
mridang f8c59d1
Align README Advanced Configuration with canonical structure
mridang 0ed69d0
Add explicit require for openssl in transport_options
mridang 1218dc4
Forward caller-provided block in factory methods
mridang 6a975d9
Add real proxy container to transport options test
mridang 7fa50cd
chore: align docs and remove inline comments
mridang e50662a
Update lib/zitadel/client/transport_options.rb
mridang d62e280
replace tinyproxy with ubuntu/squid:6.10-24.10_beta
mridang 8183a18
use unique network name to avoid collisions
mridang 858ec8d
use public _id accessor instead of instance_variable_get
mridang fb8d48e
docs: fix proxy auth docs to use URL credentials instead of default h…
mridang 83a4188
fix: add proxy container wait strategy to prevent flaky tests
mridang 0813387
fix: remove unused openssl require from client_credentials_authenticator
mridang fc5b238
fix: resolve rubocop warnings in wait_for_port
mridang 5402efb
Add missing @param transport_options YARD tags
mridang 3ffee8a
Standardize @param transport_options descriptions
mridang 0c783ab
Standardize factory method return descriptions\n\nAlign @return text …
mridang 20cac97
Standardize WireMock version to 3.12.1
mridang ccae94f
Update test/zitadel/client/transport_options_test.rb
mridang 5cdec2b
Clarify default_headers are sent to the origin server
mridang d13a05e
Replace hand-written wait_for_port with testcontainers built-in
mridang 9c35d42
Replace programmatic WireMock stubs with static JSON mapping files\n\…
mridang 465054f
Fix mapping files path and filesystem_binds argument count\n\nMove ma…
mridang d4cf583
Standardize TransportOptions docstrings for cross-SDK consistency
mridang e65caae
Restructure tests: split TransportOptions unit tests from Zitadel int…
mridang 1a93ed8
Fix rubocop offenses in transport options and zitadel tests
mridang c453172
Harden insecure precedence test with nonexistent CA cert path
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
Some comments aren't visible on the classic Files Changed page.
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
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.