[tomcat] branch main updated: Slightly more interesting test case

2021-07-08 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git The following commit(s) were added to refs/heads/main by this push: new cd6be41 Slightly more interesting test case cd6be41

Re: [tomcat] 03/03: Add support for coercing LambdaExpression to any functional interface

2021-07-08 Thread Mark Thomas
On 08/07/2021 22:05, ma...@apache.org wrote: This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git commit 2fcb104294b2676154cb08f00d5665d668792280 Author: Mark Thomas AuthorDate:

[tomcat] 03/03: Add support for coercing LambdaExpression to any functional interface

2021-07-08 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git commit 2fcb104294b2676154cb08f00d5665d668792280 Author: Mark Thomas AuthorDate: Thu Jul 8 22:02:26 2021 +0100 Add support

[tomcat] 01/03: ELSupport is a utility class. Refactor so it is accessed as one.

2021-07-08 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git commit c57c3619f75f5ddc21d79f58bdda1d0866da46cb Author: Mark Thomas AuthorDate: Thu Jul 8 20:55:20 2021 +0100 ELSupport is a

[tomcat] 02/03: Rename so strings get picked up by POEditor import/export code

2021-07-08 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git commit a37f7a11a1b644c4e9e965da730824ea656d3a3a Author: Mark Thomas AuthorDate: Thu Jul 8 21:50:04 2021 +0100 Rename so

[tomcat] branch main updated (35bd877 -> 2fcb104)

2021-07-08 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a change to branch main in repository https://gitbox.apache.org/repos/asf/tomcat.git. from 35bd877 Add more generics to EL API to align with spec project new c57c361 ELSupport is a utility class.

Re: Update "developers" list

2021-07-08 Thread Coty Sutherland
On Thu, Jul 8, 2021 at 11:17 AM Mark Thomas wrote: > On 08/07/2021 15:28, Christopher Schultz wrote: > > All, > > > > The Apache Tomcat web site has a few places where people are > > specifically listed by name. One is under "Who We Are"[1], and it's > > fairly up-to-date. (Reasonable people can

Re: Graal now supports JMX?

2021-07-08 Thread Romain Manni-Bucau
Le jeu. 8 juil. 2021 à 15:36, Rémy Maucherat a écrit : > On Thu, Jul 8, 2021 at 1:21 PM Romain Manni-Bucau > wrote: > > > > Hi Mark, > > > > My understanding of the issue is that the isGraal test is not sufficient > > and should be precised to be isGraalNative (just using the naming to > >

Re: Update "developers" list

2021-07-08 Thread Mark Thomas
On 08/07/2021 15:28, Christopher Schultz wrote: All, The Apache Tomcat web site has a few places where people are specifically listed by name. One is under "Who We Are"[1], and it's fairly up-to-date. (Reasonable people can disagree as to whether e.g. "jim" is a committer or a

[GitHub] [tomcat] ChristopherSchultz commented on pull request #428: Enhancement: Additional user attributes queried by (some) realms

2021-07-08 Thread GitBox
ChristopherSchultz commented on pull request #428: URL: https://github.com/apache/tomcat/pull/428#issuecomment-876511953 When considering defensive copying, one must remember that no other code can be trusted, but our own code _can_ be trusted. So structures passed-into the realm must be

Update "developers" list

2021-07-08 Thread Christopher Schultz
All, The Apache Tomcat web site has a few places where people are specifically listed by name. One is under "Who We Are"[1], and it's fairly up-to-date. (Reasonable people can disagree as to whether e.g. "jim" is a committer or a committer-emeritus.) There is another place people are

[GitHub] [tomcat] cklein05 commented on pull request #428: Enhancement: Additional user attributes queried by (some) realms

2021-07-08 Thread GitBox
cklein05 commented on pull request #428: URL: https://github.com/apache/tomcat/pull/428#issuecomment-876456139 On defensive copies: For my understanding, defensive copies do not prevent exposing sensitive user information to (potentially untrusted) applications due to an

Re: Graal now supports JMX?

2021-07-08 Thread Rémy Maucherat
On Thu, Jul 8, 2021 at 1:21 PM Romain Manni-Bucau wrote: > > Hi Mark, > > My understanding of the issue is that the isGraal test is not sufficient > and should be precised to be isGraalNative (just using the naming to > express the idea). > Graal is a plain JVM and this does not need any hack but

Re: Graal now supports JMX?

2021-07-08 Thread Rémy Maucherat
On Thu, Jul 8, 2021 at 1:01 PM Mark Thomas wrote: > > Hi, > > I got pinged on this issue yesterday: > https://github.com/spring-projects-experimental/spring-native/issues/805#issuecomment-875335648 > > That reads to me as if Graal now supports the use of JMX. If that is the > case I think we can

Re: Graal now supports JMX?

2021-07-08 Thread Romain Manni-Bucau
Hi Mark, My understanding of the issue is that the isGraal test is not sufficient and should be precised to be isGraalNative (just using the naming to express the idea). Graal is a plain JVM and this does not need any hack but when native-image is used it needs all current hacks so guess the test

[GitHub] [tomcat] markt-asf commented on pull request #428: Enhancement: Additional user attributes queried by (some) realms

2021-07-08 Thread GitBox
markt-asf commented on pull request #428: URL: https://github.com/apache/tomcat/pull/428#issuecomment-876355711 For defensive copies, I see several approaches: 1. GenericPrincipal always returns defensive copies 2. GenericPrincipal returns defensive copies if running under a

Graal now supports JMX?

2021-07-08 Thread Mark Thomas
Hi, I got pinged on this issue yesterday: https://github.com/spring-projects-experimental/spring-native/issues/805#issuecomment-875335648 That reads to me as if Graal now supports the use of JMX. If that is the case I think we can remove this code:

[Bug 65441] sslutils.c: function OCSP_HTTP_parse_url doesn't exist, use OSSL_HTTP_parse_url instead

2021-07-08 Thread bugzilla
https://bz.apache.org/bugzilla/show_bug.cgi?id=65441 Mark Thomas changed: What|Removed |Added Resolution|--- |FIXED Status|NEW

[tomcat-native] branch main updated: Correct fox for OpenSSL 3.x build

2021-07-08 Thread markt
This is an automated email from the ASF dual-hosted git repository. markt pushed a commit to branch main in repository https://gitbox.apache.org/repos/asf/tomcat-native.git The following commit(s) were added to refs/heads/main by this push: new 755a9cb Correct fox for OpenSSL 3.x build

[GitHub] [tomcat] rmaucher commented on pull request #428: Enhancement: Additional user attributes queried by (some) realms

2021-07-08 Thread GitBox
rmaucher commented on pull request #428: URL: https://github.com/apache/tomcat/pull/428#issuecomment-876223848 Still looks good to me, needs a super minor merging for UserDatabaseRealm. -- This is an automated message from the Apache Git Service. To respond to the message, please log on

[GitHub] [tomcat] michael-o commented on pull request #428: Enhancement: Additional user attributes queried by (some) realms

2021-07-08 Thread GitBox
michael-o commented on pull request #428: URL: https://github.com/apache/tomcat/pull/428#issuecomment-876192961 @cklein05 I am waiting for other committers to review. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use