https://issues.apache.org/bugzilla/show_bug.cgi?id=56718

Yann Ylavic <[email protected]> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[email protected]

--- Comment #3 from Yann Ylavic <[email protected]> ---
Created attachment 32455
  --> https://issues.apache.org/bugzilla/attachment.cgi?id=32455&action=edit
Fix on protocol.c and vhost.c

Thanks Regis and Tom for the report and patches.

The case is already handled by ap_update_vhost_from_headers() when
"HttpProtocol strict" is configured (though without the URI's :port being
appended).

If we were to conform to the RFC by default, which I agree with but like to ear
others about the possible compatibily caveats, I think the proposed patch are
not complete.

This new (attached) patch avoids the double work in ap_read_request() and
ap_update_vhost_from_headers(), and also takes care of IPv6-literal hostnames
(which need to be surrounded by square brackets in the Host header).

-- 
You are receiving this mail because:
You are the assignee for the bug.

---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to