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

added rex_sql options to support mysql with ssl #3311

Merged
merged 21 commits into from Feb 22, 2020
Merged

added rex_sql options to support mysql with ssl #3311

merged 21 commits into from Feb 22, 2020

Conversation

staabm
Copy link
Member

@staabm staabm commented Feb 21, 2020

closes #2196

da es vermutlich ein exotischer case ist, bewusst ohne einstellungsmöglichkeit direkt im setup, sondern nur via anpassung der config.yml

Copy link
Member

@bloep bloep left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm, ungetestet

@staabm
Copy link
Member Author

staabm commented Feb 21, 2020

@skerbis @aeberhard könnte jemand von euch sich nen mysql mit ssl zert einrichten um diesen PR hier zu testen?

siehe dazu https://dev.mysql.com/doc/refman/5.7/en/using-encrypted-connections.html

staabm and others added 3 commits February 21, 2020 22:12
using `git diff default.config.yml > ../../../.github/workflows/default.config.yml.github-action.diff` via WSL
@staabm staabm added this to the REDAXO 5.10 milestone Feb 22, 2020
@staabm staabm added the automerge Automatisch PR rebasen und mergen label Feb 22, 2020
@kodiakhq kodiakhq bot merged commit f168a10 into master Feb 22, 2020
@kodiakhq kodiakhq bot deleted the mysql-ssl branch February 22, 2020 13:52
@skerbis
Copy link
Contributor

skerbis commented Mar 6, 2021

Ich verstehe diese Konfigurationsoption überhaupt nicht. Wozu benötigt REDAXO die Pfade zu den Zertifikaten, wenn der Server doch remote ist? Die SSL/TLS Verschlüsselung braucht man allgemein ja "favorisiert" bei Remote-Zugriff.
@staabm
Diese liegt meist auch nicht in der zugreifbaren Pathlist.

@staabm
Copy link
Member Author

staabm commented Mar 6, 2021

Diese Einstellung ist dafür da, um zwischen php und mysql eine ssl/tls verschlüsselte Verbindung zu etablieren.. was genau ist unklar?

Streng genommen müsste jeder der die dsgvo ernst nimmt seine verbindung verschlüsseln weil darüber in der regel auch personen bezogene daten ausgetauscht werden

@skerbis
Copy link
Contributor

skerbis commented Mar 6, 2021

Heißt das man muss für REDAXO eigene Certs anlegen? Ich verstehe das mit der Pfadangabe aktuell nicht. Weil in anderen Systemen checke ich einfach use TLS / SSL an und es läuft.

@staabm
Copy link
Member Author

staabm commented Mar 6, 2021

Man kann tls/ssl mit und ohne zertifikaten nutzen (analog im browser gegen den webserver).

ohne zerts bekommt man eine Verschlüsselte verbindung, aber ohne sicherstellen zu können ob die gegenseite überhaupt der server ist mit dem man kommunizieren wollte.

wenn man auch zertifikate verwendet kann man die Authentizität des servers validieren, d.h. Man kann sicherstellen dass man mit dem richtigen Server kommuniziert

@skerbis
Copy link
Contributor

skerbis commented Mar 6, 2021

Man kann tls/ssl mit und ohne zertifikaten nutzen (analog im browser gegen den webserver).
Ist schon klar.

  • Ich lasse das Thema für die Doku offen - Ich kann diese Einstellung leider nicht testen. -

@skerbis
Copy link
Contributor

skerbis commented Mar 7, 2021

Doku: Aktuell hier erwähnt: https://github.com/redaxo/docs/blob/master/configyml.md

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge Automatisch PR rebasen und mergen
Development

Successfully merging this pull request may close these issues.

Mysql ssl connections unterstützen
4 participants