Re: Showcase app broken by struts.allowlist.classes

Kusal Kithul-Godage <[email protected]>
Newsgroups gmane.comp.jakarta.struts.devel
Message-ID <CAOG_=KzGuLERkztsb8uVpuTXid2Ly3Qxmq2U6ZuAJd5XqnnBXQ@mail.gmail.com>
Ah just saw this - I've already created a fix PR, took me a little
longer than expected as it turns out there were a few more broken
Showcase Actions which I've also fixed
https://github.com/apache/struts/pull/986

On Sat, Jul 13, 2024 at 7:23 PM Lukasz Lenart <[email protected]> wrote:
>
> sob., 13 lip 2024 o 10:23 Kusal Kithul-Godage
> <[email protected]> napisał(a):
> > That's correct, it's only enabled by default from 7.0, but I enabled
> > it manually for the Showcase App so we can ensure its functionality
> > and catch regressions. It seems in this case, we have an Action that
> > utilises the Convention plugin but isn't actually covered by any
> > tests.
>
> I can add a test to cover this action plus I will add @StrutsParameter
> where needed
>
> Cheers
> Lukasz
>
> ---------------------------------------------------------------------
> To unsubscribe, e-mail: [email protected]
> For additional commands, e-mail: [email protected]
>
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.