-
-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Clarify that HTTPS is not just needed for VoIP #6146
Conversation
…but for basic security.
The pr should go against the develop branch. |
Ping @rugk still have a comment not responded to. |
In addition to fixes from review, also add link to official source with more information.
Sorry, seems I've missed the notifications. 😄 Finally, improved/fixed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for rewording this
@rugk If you have time to apply the suggestion above or mark the PR so that maintainers can edit and do so, then I believe we can press the merge button after that. 😁 |
Co-Authored-By: Travis Ralston <travpc@gmail.com>
Done. FYI "Allow edits from maintainers." had already been aliowed/selected. |
for some reason it was blocking me from doing both in and out of github :( Thanks! |
…but for basic security.