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

Installer :: missachtet die Vorgaben für Datei- und Verzeichnisrechte #2225

Closed
Koala opened this issue Nov 24, 2018 · 3 comments · Fixed by #2368
Closed

Installer :: missachtet die Vorgaben für Datei- und Verzeichnisrechte #2225

Koala opened this issue Nov 24, 2018 · 3 comments · Fixed by #2368
Labels
Accepted / Help Welcome You are welcome to provide a patch/pull-request for this issue. Bug Installer "Installer"-Addon related things

Comments

@Koala
Copy link
Contributor

Koala commented Nov 24, 2018

Der Installer ignoriert die vorgegebenen Einstellungen für die Datei- und Verzeichnisrechte beim herunterladen eines Addons. Die entpackten Addons im Verzeichnis redaxo/src/addons haben somit die falschen Lese-/Schreibrechte.

Affected versions / Verwendete Versionen
REDAXO: 5.6.4
PHP: 7.0

@Koala
Copy link
Contributor Author

Koala commented Dec 10, 2018

Wenn ich das richtig sehe, müsste hier (https://github.com/redaxo/redaxo/blob/master/redaxo/src/addons/install/lib/archive.php) in der Extract-Funktion das Anpassen der Datei- und Verzeichnisrechte eingebaut werden.
Derzeit werden die Rechte nur 1:1 aus dem Archiv übernommen.

@staabm staabm added Bug Accepted / Help Welcome You are welcome to provide a patch/pull-request for this issue. labels Dec 10, 2018
@skerbis
Copy link
Contributor

skerbis commented Dec 10, 2018

@Koala gerne PR

@Koala
Copy link
Contributor Author

Koala commented Jan 1, 2019

#2368
Installation von Addons und Core-Update sind gestestet. Addon-Updates allerdings noch nicht.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted / Help Welcome You are welcome to provide a patch/pull-request for this issue. Bug Installer "Installer"-Addon related things
Development

Successfully merging a pull request may close this issue.

4 participants