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

Add access and error logs directives when www_to_non_www or non_www_to_www is used #1546

Merged
merged 2 commits into from
Jun 28, 2023

Conversation

m1keru
Copy link
Contributor

@m1keru m1keru commented Apr 6, 2023

Pull Request (PR) description

Add access_log and error_log to rewrite sections.

This Pull Request (PR) fixes the following issues

Fixes #1545

@kenyon
Copy link
Member

kenyon commented Jun 21, 2023

Please rebase on latest master to see if all of the checks pass.

@m1keru m1keru force-pushed the access_log_in_redirect_location branch from 6c0389d to 996ae5e Compare June 24, 2023 06:41
@m1keru
Copy link
Contributor Author

m1keru commented Jun 24, 2023

Please rebase on latest master to see if all of the checks pass.

Thanks, done

@kenyon kenyon changed the title Add access and error logs directives when www_to_non_www or non_www_t… Add access and error logs directives when www_to_non_www or non_www_to_www is used Jun 24, 2023
@bastelfreak
Copy link
Member

@m1keru thanks for the PR! the template is quite complex. could you add a unit test that ensures the correct content is rendered?

@m1keru m1keru force-pushed the access_log_in_redirect_location branch 5 times, most recently from e6b3482 to 39a08c9 Compare June 27, 2023 12:30
@m1keru m1keru force-pushed the access_log_in_redirect_location branch 5 times, most recently from f9e7329 to 48e5006 Compare June 27, 2023 14:21
@m1keru m1keru force-pushed the access_log_in_redirect_location branch from 48e5006 to 82e024c Compare June 27, 2023 14:26
@bastelfreak bastelfreak added the enhancement New feature or request label Jun 28, 2023
@bastelfreak bastelfreak merged commit 86918a6 into voxpupuli:master Jun 28, 2023
@m1keru
Copy link
Contributor Author

m1keru commented Jun 28, 2023

Thanks!

@m1keru m1keru deleted the access_log_in_redirect_location branch July 28, 2023 10:38
@TheMeier TheMeier added this to the 6.0.0 milestone Jun 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

access log inside rewrite_www_to_non_www and rewrite_non_www_to_www
4 participants