change PerlHandler -> PerlResponseHandler in docs #522

Merged
merged 1 commit into from Jan 12, 2014

Projects

None yet

3 participants

@Xaerxess
Contributor
Xaerxess commented Jan 9, 2014

According to http://perl.apache.org/docs/2.0/user/handlers/http.html#HTTP_Request_Cycle_Phases
"the PerlHandler directive has been renamed to PerlResponseHandler to better match the corresponding Apache phase name (response)."

Somehow the original version works on Apache 2.2.14 on Ubuntu 10.04, but fails on compiled Apache 2.2.X. I guess Ubuntu / Debian packagers are polite.

Also, @miyagawa uses PerlResponseHandler in Plack::Handler::Apache2 documentation.

@Xaerxess Xaerxess change PerlHandler -> PerlResponseHandler in docs
According to
http://perl.apache.org/docs/2.0/user/handlers/http.html#HTTP_Request_Cycle_Phases
"the PerlHandler directive has been renamed to PerlResponseHandler to
better match the corresponding Apache phase name (response)."
81ccbaf
@Xaerxess Xaerxess referenced this pull request in PerlDancer/Dancer Jan 9, 2014
Closed

change PerlHandler -> PerlResponseHandler in docs #985

@veryrusty
Member

We should not be using the (deprecated) PerlHandler directive for the Apache2 sample configuration.

@veryrusty veryrusty merged commit 81ccbaf into PerlDancer:master Jan 12, 2014

1 check passed

default The Travis CI build passed
Details
@veryrusty
Member

Merged, thanks!

@Xaerxess Xaerxess deleted the Xaerxess:pr/fix-doc-apache2-deployment branch Jan 12, 2014
@xsawyerx
Member
xsawyerx commented Mar 4, 2014

👍 Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment