[jira] [Comment Edited] (GROOVY-8186) Builder ExternalStrategy constructors have trouble with private fields

2017-05-14 Thread Paul King (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8186?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16009898#comment-16009898 ] Paul King edited comment on GROOVY-8186 at 5/15/17 1:21 AM: For case (1),

[jira] [Commented] (GROOVY-8186) Builder ExternalStrategy constructors have trouble with private fields

2017-05-14 Thread Paul King (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8186?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16009898#comment-16009898 ] Paul King commented on GROOVY-8186: --- For case (1), which is when the builder class and buildee class

[jira] [Commented] (GROOVY-7670) CLONE - dash in script filename

2017-05-14 Thread Cazacu Mihai (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-7670?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16009776#comment-16009776 ] Cazacu Mihai commented on GROOVY-7670: -- Is there any progress on this? It would be nice to have

[jira] [Comment Edited] (GROOVY-8163) Groovy scripts can disable java security manager and escape sandbox

2017-05-14 Thread Dimitry Polivaev (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8163?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16009694#comment-16009694 ] Dimitry Polivaev edited comment on GROOVY-8163 at 5/14/17 11:57 AM:

[jira] [Commented] (GROOVY-8163) Groovy scripts can disable java security manager and escape sandbox

2017-05-14 Thread Dimitry Polivaev (JIRA)
[ https://issues.apache.org/jira/browse/GROOVY-8163?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel=16009695#comment-16009695 ] Dimitry Polivaev commented on GROOVY-8163: -- Need for patching {{MetaClassImpl}} if

[GitHub] groovy pull request #532: Prevent CachedField and CachedMethod from leaking ...

2017-05-14 Thread dpolivaev
Github user dpolivaev commented on a diff in the pull request: https://github.com/apache/groovy/pull/532#discussion_r116377499 --- Diff: src/main/org/codehaus/groovy/reflection/CachedMethod.java --- @@ -124,6 +131,12 @@ public String getSignature() { }

[GitHub] groovy pull request #532: Prevent CachedField and CachedMethod from leaking ...

2017-05-14 Thread dpolivaev
Github user dpolivaev commented on a diff in the pull request: https://github.com/apache/groovy/pull/532#discussion_r116377497 --- Diff: src/main/org/codehaus/groovy/reflection/CachedMethod.java --- @@ -90,6 +91,12 @@ public CachedClass getDeclaringClass() { public

[GitHub] groovy pull request #532: Prevent CachedField and CachedMethod from leaking ...

2017-05-14 Thread dpolivaev
Github user dpolivaev commented on a diff in the pull request: https://github.com/apache/groovy/pull/532#discussion_r116377501 --- Diff: src/main/org/codehaus/groovy/reflection/CachedMethod.java --- @@ -324,6 +337,12 @@ else if (o2 instanceof CachedMethod) }

[GitHub] groovy pull request #532: Prevent CachedField and CachedMethod from leaking ...

2017-05-14 Thread dpolivaev
Github user dpolivaev commented on a diff in the pull request: https://github.com/apache/groovy/pull/532#discussion_r116377495 --- Diff: src/main/org/codehaus/groovy/reflection/CachedField.java --- @@ -65,6 +72,12 @@ public Object getProperty(final Object object) { *

[GitHub] groovy pull request #532: Prevent CachedField and CachedMethod from leaking ...

2017-05-14 Thread dpolivaev
Github user dpolivaev commented on a diff in the pull request: https://github.com/apache/groovy/pull/532#discussion_r116377482 --- Diff: src/main/org/codehaus/groovy/reflection/CachedField.java --- @@ -51,6 +52,12 @@ public int getModifiers() { */ public Object

[GitHub] groovy pull request #532: Prevent CachedField and CachedMethod from leaking ...

2017-05-14 Thread dpolivaev
Github user dpolivaev commented on a diff in the pull request: https://github.com/apache/groovy/pull/532#discussion_r116377418 --- Diff: src/main/org/codehaus/groovy/reflection/AccessPermissionChecker.java --- @@ -0,0 +1,62 @@ +/* + * Licensed to the Apache Software