[GitHub] tomcat issue #125: Provide port offset functionality (BZ-61171)

[markt-asf <gi...@git.apache.org>]

Github user markt-asf commented on the issue:

    https://github.com/apache/tomcat/pull/125
  
    Patch has been applied with a few changes:
     - portOffset was not cached on the Connector (to align with recent changes to how port is handled)
     - handle special case port values where the offset should not be applied
     - use getPortWithOffset() everywhere
     - expose  new attributes via JMX
     - take a slightly different approach with logging in case anyone (not  that they should) is parsing log messages based on the current format
     - added handling for the redirectPort attribute
     - fix any IDE / CheckStyle warnings
    
    Despite what might look like a long list of changes, the patch was applied largely as-is. I particularly like the approach to handling the Connectors.
    
    Many thanks. This feature will be available in 9.0.13 onwards.


---

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe@tomcat.apache.org
For additional commands, e-mail: dev-help@tomcat.apache.org