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

Salesforce is requiring an upgrade to TLS 1.1 or higher by July 22, 2017 #262

Open
hukllachay opened this issue Jun 27, 2017 · 4 comments
Open
Assignees

Comments

@hukllachay
Copy link

Hi

Just to confirm if the latest version already address the TLS 1.1 or higher requirement of Salesforce?

https://help.salesforce.com/articleView?id=Salesforce-disabling-TLS-1-0&language=en_US&type=1

Thank you

Limber jaimes

@avesse
Copy link
Contributor

avesse commented Jun 28, 2017

As far as I know, no. Our workaround is to call the following before calling AuthenticationClient methods:

ServicePointManager.SecurityProtocol = SecurityProtocolType.Tls12 | SecurityProtocolType.Tls11 | SecurityProtocolType.Tls;

@DavidDeSloovere
Copy link

Latest version is broken without this workaround! Using the sample WebServerOAuthFlow.Web project, just adding @avesse 's line of code in CallbackController.Get will work.

Also, the client doesn't handle the HTML response page about the TLS error because it only parses json.
https://github.com/developerforce/Force.com-Toolkit-for-NET/blob/313c4b9106b9a0f9779d42514189addc713799a5/src/CommonLibrariesForNET/AuthenticationClient.cs#L80
So you'll get a ForceAuthException saying

Unexpected character encountered while parsing value: <. Path '', line 0, position 0.

@cemerson
Copy link

Thanks for documenting this - it got me up and running again as well.

@wadewegner
Copy link
Owner

There were issues with the PCLs and including ServicePointManager. I'm hopeful that .NET Standard 2.0 will resolve this. I haven't investigated, but if I could move this into the AuthenticationClient itself it would be a big win.

@wadewegner wadewegner self-assigned this Aug 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants