sync nginx conf with upstream nextcloud docs #392
Conversation
![github-actions[bot]](https://avatars.githubusercontent.com/in/15368?s=60&v=4){kind=small}
There was a problem hiding this comment.
Thanks for opening this pull request! Be sure to follow the pull request template!
|
I am a bot, here are the test results for this PR:
|
|
I am a bot, here are the test results for this PR:
|
calvin-li-developer
changed the title
|
Please revert the fastcgi_pass/php-handler change as it would diverge from our other containers unnecessarily, but otherwise looks good and I'm happy to approve it. |
done. |
|
I am a bot, here are the test results for this PR:
|
|
Version header is duplicate |
ops forgot to delete the previous line. haha. I think @thespad fixed it. Thanks! 😊 |
| # only for Nextcloud like below: | ||
| include mime.types; | ||
| types { | ||
| text/javascript js mjs; |
There was a problem hiding this comment.
you can remove js since the mine.type already includes application/javascript js;
Description:
Updates site default conf to upstream configurations.
Benefits of this PR and context:
closes #391 and #388
How Has This Been Tested?
I have done it manually in my nextcloud instance and it seems to fix a log-reader issue and activities tab issue.
Source / References:
https://docs.nextcloud.com/server/latest/admin_manual/installation/nginx.html