Skip to content

Tomcat warns about a missing +/- prefix when enabling multiple protocols through server.ssl.enabled-protocols#36572

Closed
chicobento wants to merge 1 commit intospring-projects:mainfrom
chicobento:tomcat_ssl_protocols
Closed

Tomcat warns about a missing +/- prefix when enabling multiple protocols through server.ssl.enabled-protocols#36572
chicobento wants to merge 1 commit intospring-projects:mainfrom
chicobento:tomcat_ssl_protocols

Conversation

@chicobento
Copy link
Copy Markdown
Contributor

@chicobento chicobento commented Jul 25, 2023 •
edited
Loading

When configuring SSL with Tomcat, the following message is being logged in warning level by SSLHostConfig:

"The protocol [{0}] was added to the list of protocols on the SSLHostConfig named [{1}]. Check if a +/- prefix is missing."

This is because Tomcat's SSLHostConfig semantics for protocol string is:

        // List of protocol names, separated by ",", "+" or "-".
        // Semantics is adding ("+") or removing ("-") from left
        // to right, starting with an empty protocol set.
        // Tokens are individual protocol names or "all" for a
        // default set of supported protocols.
        // Separator "," is only kept for compatibility and has the
        // same semantics as "+", except that it warns about a potentially
        // missing "+" or "-".

@spring-projects-issues spring-projects-issues added the status: waiting-for-triage An issue we've not yet triaged label Jul 25, 2023
@wilkinsona wilkinsona changed the title Fix tomcat ssl protocol warning log Tomcat warns about a missing +/- prefix when configuring server.ssl.enabled-protocols Jul 25, 2023
@wilkinsona wilkinsona added type: bug A general bug and removed status: waiting-for-triage An issue we've not yet triaged labels Jul 25, 2023
@wilkinsona wilkinsona changed the title Tomcat warns about a missing +/- prefix when configuring server.ssl.enabled-protocols Tomcat warns about a missing +/- prefix when enabling multiple protocols through server.ssl.enabled-protocols Jul 25, 2023
@wilkinsona wilkinsona added this to the 2.7.x milestone Jul 25, 2023
@mhalbritter mhalbritter self-assigned this Jul 26, 2023
@mhalbritter mhalbritter mentioned this pull request Jul 26, 2023
Closed
mhalbritter pushed a commit that referenced this pull request Jul 26, 2023
@chicobento @mhalbritter
Fix Tomcat SSL protocol warn message
307d6c0 
See gh-36572
@mhalbritter
Copy link
Copy Markdown
Contributor

mhalbritter commented Jul 26, 2023

Thank you very much and congratulations on your first contribution 🎉!

@mhalbritter mhalbritter modified the milestones: 2.7.x, 2.7.15 Jul 26, 2023
@Netyyyy Netyyyy mentioned this pull request Aug 25, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@mhalbritter mhalbritter

Labels

type: bug A general bug

Projects

None yet

Milestone

2.7.15

Development

Successfully merging this pull request may close these issues.

4 participants

@chicobento @mhalbritter @wilkinsona @spring-projects-issues