-
Notifications
You must be signed in to change notification settings - Fork 17
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
possible upstream security issue #89
Comments
Seems it is being made configurable! For eduVPN/Let's Connect! we SHOULD set the security level to 3. |
👍 I'll update as soon as released as a CocoaPod. |
…yLevel`. This commit sets it to 3. Beware, this uses a direct git reference instead of a released version of TunnelKit. I will switch back as soon as possible. Fixes #89.
Did you test it, e.g. by setting it to 4 and 5 and trying to connect? |
Good point. I added a toggle in settings allowing selection of level 3, 4 or 5. Everything checks out. Setting level 4 or 5 does not allow connecting due to a disallowed cipher suite or key size violation. |
I would not make this configurable by the end user, but maybe you meant a toggle at build time, then that's a great idea! |
It was, but now it is not anymore. 👍 |
Keep an eye on this: passepartoutvpn/tunnelkit#97
I recommend to set this to 3 for eduVPN/Let's Connect! for iOS, possibly with a patch.
The text was updated successfully, but these errors were encountered: