[Bug 65244] annotations from @HandlesTypes are checked only at class level when scanning

2021-04-16 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=65244 --- Comment #11 from romain.manni-bucau --- Context or contextconfig configuration is fine since most integrations have listeners already no? -- You are receiving this mail because: You are the assignee for the bug. --

[Bug 65244] annotations from @HandlesTypes are checked only at class level when scanning

2021-04-16 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=65244 --- Comment #10 from Mark Thomas --- I like it. That does almost certainly mean one breaking change now to introduce the pluggable API. Would we consider ServiceLoader for this? -- You are receiving this mail because: You are the assignee for

[Bug 65244] annotations from @HandlesTypes are checked only at class level when scanning

2021-04-16 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=65244 --- Comment #9 from romain.manni-bucau --- Maybe it is an opportunity to make it properly pluggable. Most tomcat integrators drops that part to use their own scanner (tomee uses xbean, pax uses osgi flavor of xbean, others bypasses it, some use

[Bug 65244] annotations from @HandlesTypes are checked only at class level when scanning

2021-04-16 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=65244 --- Comment #8 from Mark Thomas --- I've been thinking about implementation options. It looks relatively simple although there is potential complexity depending on the extent to which we are concerned about retaining current behaviour of existi

[PROPOSAL] Change the way we present sslProtocol and sslEnabledProtocols config attributes

2021-04-16 Thread Christopher Schultz
All, The sslProtocol and sslEnabledProtocols configuration attributes on are potentially confusing to people, and there really isn't any reason for it. There is really never any reason to change sslProtocol from the default which is "TLS" because: 1. "TLS" actually enables SSLv3 when SSLv

[Bug 65244] annotations from @HandlesTypes are checked only at class level when scanning

2021-04-16 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=65244 --- Comment #7 from Remy Maucherat --- Although not urgent at all, the specification seems very clear now that I have reviewed it (annotations on fields and method do count). I'll try to do something about it next week to see how it can work.

[Bug 65240] Multi line CATALINA_OPTS is failing in with new catalina.sh

2021-04-16 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=65240 --- Comment #3 from mgrigorov --- Can you use trailing '\': -DmolnetConfigLocation=file:$CATALINA_BASE/conf/app.properties \ -Dspring.profiles.active=. \ -Dcom.sun.management.jmxremote \ -Dcom.sun.management.jmxremote.

[tomcat] branch 7.0.x updated: Fix BZ 65235. Correct docs. Expand MBean descriptor.

2021-04-16 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 7.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/7.0.x by this push: new bc50c78 Fix BZ 65235. Correct docs. Expand MBean d

[Bug 65235] Mismatch between RemoteIpValve documentation and source code

2021-04-16 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=65235 Mark Thomas changed: What|Removed |Added OS||All Resolution|---

[tomcat] branch master updated: Fix BZ 65235. Correct docs. Expand MBean descriptor.

2021-04-16 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/master by this push: new 04fbddc Fix BZ 65235. Correct docs. Expand MBean

[tomcat] branch 9.0.x updated: Fix BZ 65235. Correct docs. Expand MBean descriptor.

2021-04-16 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 9.0.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/9.0.x by this push: new 3096221 Fix BZ 65235. Correct docs. Expand MBean d

[tomcat] branch 8.5.x updated: Fix BZ 65235. Correct docs. Expand MBean descriptor.

2021-04-16 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch 8.5.x in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/8.5.x by this push: new 01770c6 Fix BZ 65235. Correct docs. Expand MBean d

[Bug 65224] JNDIRealm doesn't escape filters containing username

2021-04-16 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=65224 Mark Thomas changed: What|Removed |Added Status|NEW |RESOLVED Resolution|---