Skip to content

Specify using AllowEncodedSlashes and nocanon for httpd #7540

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

Merged

Conversation

gary-kim
Copy link
Member

@gary-kim gary-kim commented Jul 20, 2019

Closes #7471

When using wiki page names that include a slash
behind a Apache HTTPD reverse proxy,
AllowEncodedSlashes NoDecode
and appending nocanon to the ProxyPass
directive is required. This commit adds that
information to the documentation.

When using wiki page names that include a slash
behind a Apache HTTPD reverse proxy,
AllowEncodedSlashes NoDecode
and appending nocanon to the ProxyPass
directive is required. This commit adds that
information to the documentation.

Signed-off-by: Gary Kim <[email protected]>
@gary-kim gary-kim force-pushed the fix/7471/httpd-reverse-proxy-slashes-docs branch from aea589e to e623165 Compare July 20, 2019 07:31
@codecov-io
Copy link

codecov-io commented Jul 20, 2019

Codecov Report

❗ No coverage uploaded for pull request base (master@61ad048). Click here to learn what that means.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #7540   +/-   ##
=========================================
  Coverage          ?   41.24%           
=========================================
  Files             ?      469           
  Lines             ?    63620           
  Branches          ?        0           
=========================================
  Hits              ?    26238           
  Misses            ?    33959           
  Partials          ?     3423

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 61ad048...2643d14. Read the comment docs.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jul 20, 2019
@techknowlogick techknowlogick added the type/docs This PR mainly updates/creates documentation label Jul 20, 2019
@GiteaBot GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Jul 20, 2019
@GiteaBot GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jul 20, 2019
@lafriks lafriks merged commit cc384d7 into go-gitea:master Jul 20, 2019
@go-gitea go-gitea locked and limited conversation to collaborators Nov 24, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/docs This PR mainly updates/creates documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Using slash (/) in wiki title gives 404
5 participants