Cargo

HTTPS support for Tomcat 6.x is broken

Details

  • Type: Bug Bug
  • Status: Closed Closed
  • Priority: Major Major
  • Resolution: Fixed
  • Affects Version/s: 1.0-maven2
  • Fix Version/s: 1.0.4
  • Component/s: Tomcat
  • Labels:
    None
  • Environment:
    Vista SP1, JDK 6u7, Tomcat 6.0.18
  • Complexity:
    Intermediate
  • Tested on JDKs:
    JDK 6u7
  • Number of attachments :
    1

Description

HTTPS support for Tomcat 6.x is broken because the "SSLEnabled" attribute on the "Connector" element is not being set to "true"

See Tomcat docs for more details:

http://tomcat.apache.org/tomcat-6.0-doc/ssl-howto.html

  1. Text File
    CARRGO-609.patch
    12/Feb/10 9:27 AM
    0.8 kB
    Quentin Devriendt

Activity

Ascending order - Click to sort in descending order
Hide
Permalink
Quentin Devriendt added a comment -

Nobody has a workaround ?

Show
Quentin Devriendt added a comment - Nobody has a workaround ?
Hide
Permalink
Quentin Devriendt added a comment -

Patch that adds "SSLEnabled" parameter in tomcat 6 connector to support https.

http://tomcat.apache.org/migration.html#SSLEnabled%20attribute%20on%20Connector

Show
Quentin Devriendt added a comment - Patch that adds "SSLEnabled" parameter in tomcat 6 connector to support https. http://tomcat.apache.org/migration.html#SSLEnabled%20attribute%20on%20Connector
Hide
Permalink
Savas Ali Tokmen added a comment -

Fixed r1826 (2008-12-05)

Show
Savas Ali Tokmen added a comment - Fixed r1826 (2008-12-05)
Hide
Permalink
Dan Tran added a comment -

I dont think this is fixed yet, here is the content of tomcat6x/server.xml at trunk

<Connector port="@cargo.servlet.port@" protocol="HTTP/1.1"
connectionTimeout="20000"
redirectPort="8443"
scheme="@cargo.protocol@"
secure="@catalina.secure@"
debug="@catalina.logging.level@"
emptySessionPath="@catalina.connector.emptySessionPath@"
URIEncoding="@catalina.servlet.uriencoding@" />

No sigh of SSLEnabled flag

How do I reopen this?

Show
Dan Tran added a comment - I dont think this is fixed yet, here is the content of tomcat6x/server.xml at trunk <Connector port="@cargo.servlet.port@" protocol="HTTP/1.1" connectionTimeout="20000" redirectPort="8443" scheme="@cargo.protocol@" secure="@catalina.secure@" debug="@catalina.logging.level@" emptySessionPath="@catalina.connector.emptySessionPath@" URIEncoding="@catalina.servlet.uriencoding@" /> No sigh of SSLEnabled flag How do I reopen this?
Hide
Permalink
Anders Hammar added a comment -

Reopened as requested. Indeed, it does not seem to be fixed.

Show
Anders Hammar added a comment - Reopened as requested. Indeed, it does not seem to be fixed.
Hide
Permalink
Dan Tran added a comment -

is there a work around for this issue?

Show
Dan Tran added a comment - is there a work around for this issue?
Hide
Permalink
Dan Tran added a comment -

I directly patch 1.0.3's containers/tomcat with the mentioned patch and it works.!!! please apply the patch to 1.0.4-SNAPSHOT thanks

Show
Dan Tran added a comment - I directly patch 1.0.3's containers/tomcat with the mentioned patch and it works.!!! please apply the patch to 1.0.4-SNAPSHOT thanks
Hide
Permalink
Anders Hammar added a comment -

I haven't tried this as I don't work with Tomcat, but it should be easy to workaround this using Cargo 1.0.3. If you provide the server.xml you want to use in your project, you can use the configfiles element to add this file to the container's configuration.
I'm currently updating the ref guide with this info. Have a look here: http://cargo.codehaus.org/Maven2+Plugin+Reference+Guide

Show
Anders Hammar added a comment - I haven't tried this as I don't work with Tomcat, but it should be easy to workaround this using Cargo 1.0.3. If you provide the server.xml you want to use in your project, you can use the configfiles element to add this file to the container's configuration. I'm currently updating the ref guide with this info. Have a look here: http://cargo.codehaus.org/Maven2+Plugin+Reference+Guide
Hide
Permalink
Savas Ali Tokmen added a comment -

Fix for 1.0.4

PS: Tomcat 5.x and 7.x also need to be checked.

Show
Savas Ali Tokmen added a comment - Fix for 1.0.4 PS: Tomcat 5.x and 7.x also need to be checked.
Hide
Permalink
Savas Ali Tokmen added a comment -

Committed revision 2482, fixed for both Tomcat 6.x and 7.x.

Show
Savas Ali Tokmen added a comment - Committed revision 2482, fixed for both Tomcat 6.x and 7.x.
Hide
Permalink
Dan Tran added a comment -

thanks for quick fix and alternative solution

Show
Dan Tran added a comment - thanks for quick fix and alternative solution

People

  • Assignee:
    Savas Ali Tokmen
    Reporter:
    Wyatt Tellis
Vote (2)
Watch (2)

Dates

  • Created:
    Updated:
    Resolved: