Skip to content

Conversation

@millerthegorilla
Copy link
Contributor

added code to test for existence of unauthorized.log and create it if not exists.
/config/log/nginx/unauthorized.log is written to by addition of nginx-unauthorized jail in jail.local

linuxserver.io


  • [ x] I have read the contributing guideline and understand that I have made the correct modifications

Description:

Benefits of this PR and context:

This fixes a bug where the current image complains repeatedly that it is unable to find the file 'unauthorized.log'

How Has This Been Tested?

image has been built locally.

Source / References:

added code to test for existence of unauthorized.log and create it if not exists.
/config/log/nginx/unauthorized.log is written to by addition of nginx-unauthorized jail in jail.local at (https://github.com/linuxserver/docker-swag/blob/master/root/defaults/jail.local)
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for opening this pull request! Be sure to follow the pull request template!

@LinuxServer-CI
Copy link
Contributor

@aptalca
Copy link
Member

aptalca commented Mar 28, 2022

Thanks for the PR. Can you add a changelog entry as well, to readme-vars.yml?

@millerthegorilla
Copy link
Contributor Author

I have made a separate pull request to add the changelog.

added changelog
@millerthegorilla
Copy link
Contributor Author

Ok, I have added the changelog to the file. Sorry I didn't do it earlier.

Copy link
Member

@aptalca aptalca left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good, thanks

@LinuxServer-CI
Copy link
Contributor

@aptalca aptalca merged commit 5beb4ff into linuxserver:master Mar 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants