Re: [GitHub] [struts] lukaszlenart merged pull request #496: fix double evaluations

2022-03-17 Thread Lukasz Lenart
czw., 17 mar 2022 o 13:10 Yasser Zamani  napisał(a):
> > Yasser, can you add such notes to the Internal Changes section in the
> > Version Notes?
> > https://cwiki.apache.org/confluence/display/WW/Version+Notes+2.5.30
>
> Yes sure thanks for heads up! done :)

Great, thanks a lot and thanks for your patience :)


Regards
-- 
Łukasz
+ 48 606 323 122 http://www.lenart.org.pl/

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



Re: [GitHub] [struts] lukaszlenart merged pull request #496: fix double evaluations

2022-03-17 Thread Yasser Zamani




On 3/17/2022 9:47 AM, Lukasz Lenart wrote:

śr., 16 mar 2022 o 14:02 Yasser Zamani  napisał(a):


Hi Greg,

Thanks for asking and testing!

Firstly simply run all your app tests, you shouldn't see any WARN log
like "Expression blahblah isn't allowed by pattern blahblah! See
Accepted / Excluded patterns at
https://struts.apache.org/security/;.

Secondly see if your following components are functioning correctly
regarding java-scripts:
forms with client side validations
doubleselect
combobox

Also check your StreamResults, AliasInterceptors and JasperReportResults
if you have.


Yasser, can you add such notes to the Internal Changes section in the
Version Notes?
https://cwiki.apache.org/confluence/display/WW/Version+Notes+2.5.30




Yes sure thanks for heads up! done :)

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



Re: [GitHub] [struts] lukaszlenart merged pull request #496: fix double evaluations

2022-03-17 Thread Lukasz Lenart
śr., 16 mar 2022 o 14:02 Yasser Zamani  napisał(a):
>
> Hi Greg,
>
> Thanks for asking and testing!
>
> Firstly simply run all your app tests, you shouldn't see any WARN log
> like "Expression blahblah isn't allowed by pattern blahblah! See
> Accepted / Excluded patterns at
> https://struts.apache.org/security/;.
>
> Secondly see if your following components are functioning correctly
> regarding java-scripts:
> forms with client side validations
> doubleselect
> combobox
>
> Also check your StreamResults, AliasInterceptors and JasperReportResults
> if you have.

Yasser, can you add such notes to the Internal Changes section in the
Version Notes?
https://cwiki.apache.org/confluence/display/WW/Version+Notes+2.5.30


Thanks in advance
-- 
Łukasz
+ 48 606 323 122 http://www.lenart.org.pl/

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



Re: [GitHub] [struts] lukaszlenart merged pull request #496: fix double evaluations

2022-03-16 Thread Yasser Zamani

Hi Greg,

Thanks for asking and testing!

Firstly simply run all your app tests, you shouldn't see any WARN log 
like "Expression blahblah isn't allowed by pattern blahblah! See 
Accepted / Excluded patterns at

https://struts.apache.org/security/;.

Secondly see if your following components are functioning correctly 
regarding java-scripts:

forms with client side validations
doubleselect
combobox

Also check your StreamResults, AliasInterceptors and JasperReportResults 
if you have.


Thanks in advance!

Regards.

On 3/16/2022 12:33 PM, Greg Huber wrote:

Seems a lot in here, what should we look out for in testing?

On 15/03/2022 13:33, GitBox wrote:

lukaszlenart merged pull request #496:
URL: https://github.com/apache/struts/pull/496






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



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



Re: [GitHub] [struts] lukaszlenart merged pull request #496: fix double evaluations

2022-03-16 Thread Greg Huber

Seems a lot in here, what should we look out for in testing?

On 15/03/2022 13:33, GitBox wrote:

lukaszlenart merged pull request #496:
URL: https://github.com/apache/struts/pull/496








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



[GitHub] [struts] lukaszlenart merged pull request #496: fix double evaluations

2022-03-15 Thread GitBox


lukaszlenart merged pull request #496:
URL: https://github.com/apache/struts/pull/496


   


-- 
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: dev-unsubscr...@struts.apache.org

For queries about this service, please contact Infrastructure at:
us...@infra.apache.org



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