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

Update SMSController.cs #38

Closed
wants to merge 1 commit into from
Closed

Conversation

caseydierking
Copy link

Due to TLS upgrade, this line is needed to make examples work.

See this issue.

Due to TLS upgrade, this line is needed to make examples work.
@caseydierking
Copy link
Author

It'd be nice to know why this was closed without any explanation...
Is something wrong? Something need to be changed?

A little off putting for a first time contributor to nexmo-community.

@RabebOthmani
Copy link
Contributor

Hi @caseydierking
Sorry for not providing any explanations. Your contributions are always welcome and very useful to us.
We do not actually support TLSv1.1 , the protocol needs to be v1.2.

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