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

SSL fixes and additions #49

Merged
merged 11 commits into from Jun 15, 2017
Merged

Conversation

allanbartley
Copy link
Contributor

Added mention of SHA1 in docs.
Fixed inconsistent newline chars in keywords.txt.
🐛 In fingerprint check fingerprint is now a non-pointer type, fixed referencing.
🐛 In disconnection handler _clear() is called and clears _tlsBadFingerprint and _connectPacketNotEnoughSpace before they are tested. Moved _clear() call further down in function.

Added function to set value of secure.
Altered connect function to pass secure variable.
All modifications depend on ASYNC_TCP_SSL_ENABLED.
…g connection.

setServerFingerprint may be called multiple times to set more than one permitted value.
Added note about platform for PlatformIO.
Renamed setServerFingerprint to addServerFingerprint.
Corrected problem with disconnection on bad fingerprint.
Variable fingerprint is now a non-pointer type.
Added array initializer and function call outputs to get-fingerprint script.
Existing arrangement had _tlsBadFingerprint and _connectPacketNotEnoughSpace cleared before they were being checked.
@allanbartley
Copy link
Contributor Author

Also, added additional outputs to the get-fingerprint script.
Now prints additional SHA1 values in ready-to-copy formats which can be pasted straight into code.

@marvinroger marvinroger merged commit 5b4f5bf into marvinroger:master Jun 15, 2017
@marvinroger
Copy link
Owner

Awesome, thanks 😇

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.

None yet

2 participants