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

Update _default.yml #8

Closed
wants to merge 1 commit into from
Closed

Update _default.yml #8

wants to merge 1 commit into from

Conversation

rssnyder
Copy link
Contributor

@rssnyder rssnyder commented May 3, 2023

for your consideration...

reading the tinyproxy man page, the default locations tinyproxy checks for configuration is: /etc/tinyproxy/tinyproxy.conf, /var/run/tinyproxy/tinyproxy.pid, /var/log/tinyproxy/tinyproxy.log

When running this playbook on an OS that does not have a specific configuration listed in the vars folder, the config file is created at a location that tinyproxy when run as a service, is not picked up, and the service runs with the default configuration that comes with the package install.

is there some other reason the default is in this location? otherwise, I think we need to merge #3 so that it can be overridden by users.

@tersmitten
Copy link
Member

tersmitten commented May 10, 2023

I think it's there because I found it there in the packages. Not sure if it's the best one tough. I'll have a look.

@tersmitten tersmitten self-assigned this May 10, 2023
@tersmitten tersmitten added the question Further information is requested label May 10, 2023
@rssnyder rssnyder closed this Jul 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants