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

Nginx Plugin Logging Issues When /var/log is Mounted as RAM Disk #4186

Open
3 tasks done
OoSTARTERoO opened this issue Aug 19, 2024 · 3 comments · May be fixed by #4199
Open
3 tasks done

Nginx Plugin Logging Issues When /var/log is Mounted as RAM Disk #4186

OoSTARTERoO opened this issue Aug 19, 2024 · 3 comments · May be fixed by #4199

Comments

@OoSTARTERoO
Copy link

Important notices
Before you add a new report, we ask you kindly to acknowledge the following:

This is continuation of issue#4162
I have identified the root cause of the problem in more detail

Describe the bug
After a reboot, Nginx should create a log folder "/var/log/nginx" with permission of 755
Instead, The folder is created with a permission of 700 with a owner of "root:staff"

Since Nginx is running with user "www"
It cannot write any files to the /var/log/nginx

To Reproduce
System > Settings > Misc
Enable /var/log RAM Disk
Reboot.

Expected behavior
/var/log folder should be created with permission of 755.

Environment

OPNsense 24.7.1 (amd64).
Nginx plugin 1.34
with /var/log RAM Disk enabled.

@kulikov-a
Copy link
Member

kulikov-a commented Aug 25, 2024

Hi
can you try with
opnsense-patch -c plugins -a kulikov-a 250e8e9 (kulikov-a@250e8e9) ?
(need to "Apply" nginx config after that)

@OoSTARTERoO
Copy link
Author

@kulikov-a
Yes! I've just tried it, and it is working flawlessly!
(I applied the patch, and pressed "apply" on nginx. After that I did a full reboot.)

Thanks a lot!

@kulikov-a
Copy link
Member

great, thanks! i'll make a pr

@kulikov-a kulikov-a linked a pull request Aug 25, 2024 that will close this issue
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

2 participants