Tomasz Adamski

hi @darranl could you please take a look whether such refactor makes sense? The goal is to be able to migrate EARs with IronJacamar configuration files to Elytron.

@emmartins Could you please review whether this fix makes sense from naming perspective?

just noticed we have to hold it because of https://github.com/wildfly/wildfly/pull/16107/files

updated

@darranl I think we can close it; I still need to do some cleaning here but I'm working on it in another branches currently

> @tadamski does this logic runs for every JNDI lookup? Yes, every lookup from within a server is wrapped so I was able to plug verification interceptors. > This appears...

@TomasHofman fixed

> This approach seems very hacky to me. Have you considered decorating the org.wildfly.naming.client.NamingContextFactory implementation from wildfly-transaction-client? That way you can add your access check to before delegating to the...

@darranl I think the issue is correct. IIRC there was an error in appclient tests which tried to extract property from nonexistent subsystem. I proposed this fix https://github.com/wildfly/wildfly/pull/15418, but from...

now there is some other test failing but tbh I don't expect it to be related