On 09/12/2021 14:04, Mark Thomas wrote:
On 09/12/2021 11:15, Rémy Maucherat wrote:
On Thu, Dec 9, 2021 at 11:54 AM Mark Thomas <ma...@apache.org> wrote:

On 09/12/2021 10:39, 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


The following commit(s) were added to refs/heads/main by this push:
       new bb432a0  Fix BZ 65714 - previous fix was incomplete.
bb432a0 is described below

commit bb432a0fea83f424b013267ba7675c5d636a465a
Author: Mark Thomas <ma...@apache.org>
AuthorDate: Thu Dec 9 10:24:07 2021 +0000

      Fix BZ 65714 - previous fix was incomplete.

      https://bz.apache.org/bugzilla/show_bug.cgi?id=65714

Something I don't yet understand is going on here.

I am fairly sure this was working after the first fix.

I test it today and it isn't. Fair enough. I pre-load a few more classes
and that appears to fix it so I back-port the changes.

Not sure why but I decided to retest after the back-ports and it is
failing again.

I'm investigating now. This is mainly an heads-up that further fixes are
likely.

Hmmm, weird indeed. I didn't try to look at it again. OTOH, we could
just say "not supported" since there's an easy alternative and the
security manager is going away.

I've reverted today's commit as is clearly isn't correct. I'm still trying to figure out what happened earlier today and why I thought the previous fix was sufficient.

Got to the bottom of this at least. JDK differences. It appears to be differences in the Temurin JDK between MacOS and Linux. Everything is fine on MacOS which is what I was working on last week when I fixed this the first time.

Mark

I'm seeing quite a few errors on InnocuousThread instances. Those threads are designed not to have a security context. If NIO2 IO is using those, getting things working under a security manager will be much harder.

I'm beginning to lean towards your "Use NIO" solution.

Mark

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



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

Reply via email to