>-----Original Message----- >From: Lukasz Lenart <lukaszlen...@apache.org> >Sent: Wednesday, January 23, 2019 11:34 AM >To: Struts Users Mailing List <user@struts.apache.org> >Subject: Re: Java 11 support > >śr., 23 sty 2019 o 08:35 Yasser Zamani <yasserzam...@apache.org> napisał(a): >> I would prefer to allow user to optionally override the default Opcodes.ASM5 >via a constant in struts.xml e.g. <constant name="struts.convention.asmapi" >value=" Opcodes.ASM7"/>, Or more better, to think if it's possible to >determine it >at runtime via user's runtime context e.g. java version. Wdyt? > >I would start with detecting a proper Java version first. The additional >constant >will introduce unneeded burden for users to control it.
Fortunately it seems it isn't required. Locally all tests are passing here with all java versions (7,8,9,11) for [1]. Kind Regards. [1] https://github.com/apache/struts/pull/315 "According to ASM7's source codes, it seems it uses passed api version only to check if it should throw an unsupported exception or not (i.e. doesn't have any effect on functionality and somehow is just a validation). So, as we upgrade to asm.version=7.0 for default then I think it's OK to upgrade Opcodes.ASM5 to Opcodes.ASM7 as well. It only breaks when user manually exclude ASM7 dependency with an older version which is not usually expected." --------------------------------------------------------------------- To unsubscribe, e-mail: user-unsubscr...@struts.apache.org For additional commands, e-mail: user-h...@struts.apache.org
