Re: [GitHub] struts pull request: WW-4540: Strict DMI

2015-09-17 Thread Greg Huber
For my form bean, getBean().getName(); edit!getBean().getName For me it shows an exception with the bean Name field value, would a combination of all public methods in the package and then on sensitive actions like login/payments etc use the action to restrict to allowed methods only? [ edit!

Re: [GitHub] struts pull request: WW-4540: Strict DMI

2015-09-17 Thread Lukasz Lenart
2015-09-17 9:11 GMT+02:00 Greg Huber : > For my form bean, getBean().getName(); > > > edit!getBean().getName > > > For me it shows an exception with the bean Name field value, would a > combination of all public methods in the package and then on sensitive > actions like login/payments etc use the

Re: [GitHub] struts pull request: WW-4540: Strict DMI

2015-09-17 Thread Greg Huber
I was testing using: and it returned the value in the name field on the bean. public class EventEdit extends EventBase { private EventBean bean = null; /** * Gets the bean. * * @return the bean */ public EventBean getBean() { return bean; } }

Re: [GitHub] struts pull request: WW-4540: Strict DMI

2015-09-17 Thread Lukasz Lenart
2015-09-17 9:37 GMT+02:00 Greg Huber : > I was testing using: > > > > and it returned the value in the name field on the bean. > > public class EventEdit extends EventBase { > > private EventBean bean = null; > > /** > * Gets the bean. > * > * @return the bean > */ >

[GitHub] struts pull request: Improve LocalizedTextUtil.getMessage() defend...

2015-09-17 Thread quaff
Github user quaff commented on the pull request: https://github.com/apache/struts/pull/50#issuecomment-141288110 It's up-to-date on branch master. you can check commit history. https://github.com/quaff/struts/commits/master https://github.com/apache/struts/commits/master --- I

Build failed in Jenkins: Struts-JDK7-pull-request #64

2015-09-17 Thread Apache Jenkins Server
See -- GitHub pull request #50 to apache/struts [EnvInject] - Loading node environment variables. Building remotely on H10 (Ubuntu ubuntu) in workspace

[GitHub] struts pull request: Improve LocalizedTextUtil.getMessage() defend...

2015-09-17 Thread lukaszlenart
Github user lukaszlenart commented on the pull request: https://github.com/apache/struts/pull/50#issuecomment-141344975 hm... but Jenkins has a problem with building this PR :\ --- If your project is set up for it, you can reply to this email and have your reply appear on GitHub as we