Skip to content
This repository has been archived by the owner on Jan 9, 2023. It is now read-only.

Alias /error_handler to /cgi-bin so that ErrorDocuments are found #1746

Merged
merged 3 commits into from
Jun 3, 2020

Conversation

tst-jlarocco
Copy link
Contributor

@tst-jlarocco tst-jlarocco commented May 22, 2020

Fixes #1742.

@googlebot googlebot added the cla: yes Manual verification that all contributors have signed the CLA. label May 22, 2020
@tst-jlarocco tst-jlarocco reopened this May 22, 2020
# directives as to Alias.
#
ScriptAlias /cgi-bin/ "@exp_cgidir@/"
+ Alias /error_handler "/opt/google/gehttpd/cgi-bin/"
Copy link
Collaborator

Choose a reason for hiding this comment

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

Minor: could this be "/cgi-bin/" instead of the full path ("/opt/google/gehttpd/cgi-bin/"`). That seems a bit cleaner to me, but it's not a big deal.

Copy link
Collaborator

@tst-lsavoie tst-lsavoie left a comment

Choose a reason for hiding this comment

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

I tested this with several errors and all were handled as expected.

@tst-lsavoie tst-lsavoie merged commit 4d781fa into google:master Jun 3, 2020
tst-cjeffries pushed a commit to tst-cjeffries/earthenterprise that referenced this pull request Sep 14, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
cla: yes Manual verification that all contributors have signed the CLA.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Mis-configured Apache ErrorDocuments
4 participants