nsoft commented on PR #2017:
URL: https://github.com/apache/shiro/pull/2017#issuecomment-2729937167

   One note from using this patch locally: It's now the case that the faces 
stuff in shiro-jakarta-ee requires omnifaces and omnifaces is very rude if the 
container finds the annotated classes and tries to do something with them...
   ![Screenshot from 2025-03-17 
11-03-58-cdi-nope](https://github.com/user-attachments/assets/021b9a95-e3f8-4d17-87ce-8f712791c7cd)
   
   Luckily this library was something I included long ago in my early attempts 
to get ee stuff working and not actually needed AFAICT, but breaking the 
jakarta-ee stuff into smaller bits (at leaast one to separate out the omnifaces 
dependency might be a good idea (but probably a follow on ticket). 
   
   For the purposes of this ticket I'm not sure if I might have introduced that 
dep to get this working or if there might be a more polite alternative 
implementation, so that should be checked on.


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to