[go: nahoru, domu]

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

Missing documentation for creating twig cache dir #13226

Closed
nijel opened this issue Apr 28, 2017 · 5 comments · Fixed by #13237
Closed

Missing documentation for creating twig cache dir #13226

nijel opened this issue Apr 28, 2017 · 5 comments · Fixed by #13237
Assignees
Milestone

Comments

@nijel
Copy link
Contributor
nijel commented Apr 28, 2017

As we started to use twig templates, the cache dir needs to be properly set up. This is not really covered by the documentation it should at least cover:

  • secure permissions for the cache dir
  • blocking access from browser to the cache
  • moving the directory out of www root
@nijel nijel added this to the 4.8.0 milestone Apr 28, 2017
@udan11
Copy link
Contributor
udan11 commented Apr 29, 2017

Regarding "blocking access from browser to the cache", wouldn't it be better if we provided a .htaccess that does that?

@nijel
Copy link
Contributor Author
nijel commented Apr 29, 2017

There are also other web servers then Apache :-). Also Apache is often configured not to process .htaccess files, that's why we have removed them some time ago...

@udan11
Copy link
Contributor
udan11 commented Apr 29, 2017

Yes, I am aware. It's just Apache has quite a big market share. Anyway, since the other ones were removed I see no point in introducing them again.

@nijel
Copy link
Contributor Author
nijel commented May 4, 2017

Problem was that our documentation has relied on working .htaccess files. Maybe it would make sense to ship them, but still cover this topic in the documentation to cover users where .htaccess is not working. On the other side enabling .htaccess is generally not recommended by Apache docs....

@nijel nijel mentioned this issue May 4, 2017
@nijel nijel self-assigned this May 4, 2017
@nijel
Copy link
Contributor Author
nijel commented May 4, 2017

When looking at this, I've realized that it might be better to just use existing TempDir for twig cache and we have already quite good docs on that...

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Jun 22, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants