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

Install: different way to check app/courses/ is writeable #2601

Closed
ywarnier opened this issue Jul 21, 2018 · 2 comments
Closed

Install: different way to check app/courses/ is writeable #2601

ywarnier opened this issue Jul 21, 2018 · 2 comments

Comments

@ywarnier
Copy link
Member

Current behavior

When installing, a test directory is created in app/courses/XxTESTxX (+ a file "test.php" in it) to check if we have the right permissions and if the file is accessible and executable through the web.
However, this is now getting blocked by the recommended security settings in documentation/security.html which prevent execution of PHP from app/courses/

Expected behavior

In recent versions, it is not necessary anymore to execute PHP code from app/courses, so the check of execution doesn't make sense anymore either.
We should only check the creation of the directory and access to a simple .html file.

Steps to reproduce

  • configure security settings from security.html
  • restart web server
  • try to install chamilo
  • creation of test dir fails (because of check_course_script_interpretation() in install.lib.php)

Chamilo Version

1.11.x 2018-07-21

@jmontoyaa
Copy link
Member

I confirm the bug, it should be fixed now.

@ywarnier
Copy link
Member Author

Looks all good to me. Tested and approved. Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants