----- Original Message -----
From: "Remy Maucherat" <[EMAIL PROTECTED]>
To: "Tomcat Developers List" <[EMAIL PROTECTED]>
Sent: Sunday, September 29, 2002 10:31 AM
Subject: Re: cvs commit:
jakarta-tomcat-connectors/coyote/src/java/org/apache/coyote/tomcat4
CoyoteAdapter.java


> [EMAIL PROTECTED] wrote:
> > nacho       2002/09/29 10:07:44
> >
> >   Modified:    coyote/src/java/org/apache/coyote/tomcat4
CoyoteAdapter.java
> >   Log:
> >   Bug#12998 HTTPS gets changed to HTTP://servername:443
> >   Reported by marcus.kellermann at bentley.com
> >
> >   The processor (HTTP11 or ajp13) should set the scheme and port prior
> >   to this point, in an ajp13 connection doesnt make sense to get the
secure
> >   flag from the connector secure flag.
> >
>
> -1 for this unless you explain. The idea was to redirect to the default
> port for the protocol. Bill can probably give additional information
> (ignore the -1 if he agrees with the change).
>

Sorry, I've been out all day.

At the moment, for the HTTP11 connector, the Processor sets the ServerPort
and the Scheme correctly (even for the Servlet 2.4 spec :).  Nacho's patch
shouldn't break anything.  IMHO, low-level things like ServerPort and Scheme
belong in the Processor, and not in the Adapter.

I'll give my +1 to Nacho's patch.

> Remy
>
>
> --
> To unsubscribe, e-mail:
<mailto:[EMAIL PROTECTED]>
> For additional commands, e-mail:
<mailto:[EMAIL PROTECTED]>
>
>


--
To unsubscribe, e-mail:   <mailto:[EMAIL PROTECTED]>
For additional commands, e-mail: <mailto:[EMAIL PROTECTED]>

Reply via email to