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.

Sign up for GitHub

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

Jump to bottom

set tls version via chisel env vars #241

Open
wants to merge 1 commit into
base: master
Choose a base branch
from
Open

set tls version via chisel env vars #241

wants to merge 1 commit into from

Conversation

mikegarts
Copy link

No description provided.

@mikegarts mikegarts mentioned this pull request Jan 31, 2021
Open
@jpillora
Copy link
Owner

jpillora commented Feb 18, 2021 •
edited
Loading

Thanks for the PR :) Confirming, the user can set either of the two env vars to "1.0" / "1.1" / "1.2" / "1.3" ?

@mikegarts
Copy link
Author

Thanks for the PR :) Confirming, the user can set either of the two env vars to "1.0" / "1.1" / "1.2" / "1.3" ?

Yes, for example:
CHISEL_TLS_MAX_VER=1.2 CHISEL_TLS_MIN_VER=1.3 ./chisel client -v ....
2021/02/28 17:00:52 client: Setting tlsConfig.MinVersion to 0x304
2021/02/28 17:00:52 client: Setting tlsConfig.MaxVersion to 0x303

@mikegarts
Copy link
Author

Hi @jpillora, is there any changes you'd like me to do for the feature?
Thanks,
Michael.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mikegarts @jpillora