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

Allow HTTPS endpoints in client #338

Merged
merged 2 commits into from
Oct 25, 2021
Merged

Conversation

chen-anders
Copy link

There may be times when toxiproxy is behind an TLS-terminated endpoint; we shouldn't stop someone from setting such an endpoint when initializing a new client.

@chen-anders chen-anders requested a review from miry October 25, 2021 05:53
@chen-anders chen-anders force-pushed the anders/support-https-endpoints branch from dbf5b1c to 6178848 Compare October 25, 2021 05:54
@miry
Copy link
Contributor

miry commented Oct 25, 2021

@chen-anders Pls update Changelog.md as well.

@miry miry mentioned this pull request Oct 25, 2021
9 tasks
@chen-anders chen-anders merged commit fc4a464 into master Oct 25, 2021
@chen-anders chen-anders deleted the anders/support-https-endpoints branch October 25, 2021 13:52
@miry miry added this to the 2.3.0 milestone Nov 6, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants