-
-
Notifications
You must be signed in to change notification settings - Fork 38
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
Encrypt transfer data #54
Comments
Absolutely, wanna do a pull request? |
Would we close it in times of HTTPS und free letsencrypt certificates? |
I would rather force HTTPS usage per setting in EM, what so you think? |
I don't think, closing this is the best idea. Because having an https connection allows an attacker knowing your secret to get the data fetched. |
Additional encryption is more secure. But the attack vector you described, is easily mitigated by employing a proper IP filter in the extension's settings. |
I think it would be nice to encrypt the data transferred between monitor and clients based on a pre-shared key in the extension-settings like Xavier Perseguers does in his "Central account management":
https://docs.typo3.org/typo3cms/extensions/causal_accounts/AdministratorManual/InstallingExtension/Index.html
The text was updated successfully, but these errors were encountered: