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

--access-log option overwritten everytime #4

Merged
merged 1 commit into from
Mar 21, 2014
Merged

--access-log option overwritten everytime #4

merged 1 commit into from
Mar 21, 2014

Conversation

bndr
Copy link
Contributor

@bndr bndr commented Mar 21, 2014

Use the log file from config only when not supplied with --access-log option, else it is overwritten everytime, even when --access-log option is provided.

Use the log file from config only when not supplied with --access-log option, else it is overwritten everytime, even when --access-log option is provided.
lebinh added a commit that referenced this pull request Mar 21, 2014
--access-log option overwritten everytime
@lebinh lebinh merged commit 87ca10d into lebinh:master Mar 21, 2014
@lebinh
Copy link
Owner

lebinh commented Mar 21, 2014

thanks, I put the log_file and log_format params just for this :)
Anyway, that function really needs to be rewritten to handle more general case, it's quite limited now.

@bndr
Copy link
Contributor Author

bndr commented Mar 21, 2014

Yeah, I saw your TODO comments.

Still a nice job!

@bndr bndr deleted the patch-1 branch March 21, 2014 17:36
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.

None yet

2 participants