GitHub user sdutry opened a pull request:
https://github.com/apache/struts/pull/110
updated javadoc after changing to enum
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sdutry/struts WW-4686-javadoc
Alternatively you can
GitHub user sdutry opened a pull request:
https://github.com/apache/struts/pull/111
replaced String constants for marking status with enum
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sdutry/struts CycleDetectorEnum
Github user sdutry commented on the issue:
https://github.com/apache/struts/pull/111
registerd jira issue
[WW-4689](https://issues.apache.org/jira/browse/WW-4689)
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your
GitHub user sdutry opened a pull request:
https://github.com/apache/struts/pull/112
WW-4690 added version to jshint-maven-plugin
[issue WW-4690](https://issues.apache.org/jira/browse/WW-4690)
You can merge this pull request into a Git repository by running:
$ git pull https
GitHub user sdutry opened a pull request:
https://github.com/apache/struts/pull/126
WW-4770 Fixes problem with dojo.version
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sdutry/struts issue/WW-4770
Alternatively you can
GitHub user sdutry opened a pull request:
https://github.com/apache/struts-extras/pull/3
added module for extra result types
This is still work in progress.
The reason i'm already having it as pull request is because any input is
welcome.
related to WW
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts-extras/pull/3#discussion_r108343668
--- Diff:
struts2-custom-results-plugin/src/main/java/org/apache/struts2/result/HttpsOffloadAwareServletRedirectResult.java
---
@@ -0,0 +1,145
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts-extras/pull/3#discussion_r108343998
--- Diff:
struts2-custom-results-plugin/src/main/java/org/apache/struts2/result/HttpsOffloadAwareServletRedirectResult.java
---
@@ -0,0 +1,145
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts-extras/pull/3#discussion_r108344564
--- Diff:
struts2-custom-results-plugin/src/main/java/org/apache/struts2/result/HttpsOffloadAwareServletRedirectResult.java
---
@@ -0,0 +1,145
Github user sdutry commented on the issue:
https://github.com/apache/struts-extras/pull/3
@lukaszlenart
I think i might be about where i would end up without extra input.
Could you (or anyone else) please do another review.
Seeing as the issue where this idea
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts-extras/pull/3#discussion_r108511395
--- Diff: struts2-custom-results-plugin/README.md ---
@@ -0,0 +1,42 @@
+# Apache Struts 2 Extras - custom results plugin
+
+[![License](http
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts-extras/pull/3#discussion_r108515221
--- Diff: struts2-custom-results-plugin/pom.xml ---
@@ -0,0 +1,104 @@
+
+
+http://maven.apache.org/POM/4.0.0";
xmlns:xsi="http://
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts-extras/pull/3#discussion_r108515260
--- Diff: struts2-custom-results-plugin/pom.xml ---
@@ -0,0 +1,104 @@
+
+
+http://maven.apache.org/POM/4.0.0";
xmlns:xsi="http://
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts-extras/pull/3#discussion_r108515374
--- Diff: struts2-custom-results-plugin/pom.xml ---
@@ -0,0 +1,104 @@
+
+
+http://maven.apache.org/POM/4.0.0";
xmlns:xsi="http://
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts-extras/pull/3#discussion_r108515624
--- Diff:
struts2-custom-results-plugin/src/main/java/org/apache/struts2/result/SslOffloadAwareServletActionRedirectResult.java
---
@@ -0,0 +1,123
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts-extras/pull/3#discussion_r108515681
--- Diff:
struts2-custom-results-plugin/src/main/java/org/apache/struts2/result/SslOffloadAwareServletRedirectResult.java
---
@@ -0,0 +1,188
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts-extras/pull/3#discussion_r108516053
--- Diff:
struts2-custom-results-plugin/src/main/java/org/apache/struts2/result/SslOffloadAwareServletRedirectResult.java
---
@@ -0,0 +1,188
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts-extras/pull/3#discussion_r108519883
--- Diff: struts2-custom-results-plugin/README.md ---
@@ -0,0 +1,42 @@
+# Apache Struts 2 Extras - custom results plugin
+
+[ that i didn't break the builds before committing.
---
If your pr
GitHub user sdutry opened a pull request:
https://github.com/apache/struts/pull/136
WW-4793 only add JBossFileManager when supported
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sdutry/struts WW-4793
Alternatively you can
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts/pull/136#discussion_r114154579
--- Diff:
core/src/main/java/org/apache/struts2/util/fs/JBossFileManager.java ---
@@ -210,4 +187,33 @@ private void addIfAbsent(List urls, URL fileUrl
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts/pull/136#discussion_r114157280
--- Diff:
core/src/main/java/org/apache/struts2/util/fs/JBossFileManager.java ---
@@ -210,4 +187,33 @@ private void addIfAbsent(List urls, URL fileUrl
Github user sdutry closed the pull request at:
https://github.com/apache/struts/pull/136
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is
Github user sdutry commented on the issue:
https://github.com/apache/struts/pull/136
Given the input on the issue, this pull request should not be merged.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project
GitHub user sdutry opened a pull request:
https://github.com/apache/struts/pull/149
WW-4821 Upgrade Commons IO to 2.5
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sdutry/struts WW-4821
Alternatively you can review and apply
GitHub user sdutry opened a pull request:
https://github.com/apache/struts/pull/150
WW-4819 fixed commons-collections groupId in dependency management
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sdutry/struts WW-4819
GitHub user sdutry opened a pull request:
https://github.com/apache/struts/pull/151
WW-4818 change default Multipart validation regex to comply with RFC1341
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sdutry/struts WW-4818
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts/pull/151#discussion_r129281083
--- Diff: core/src/main/java/org/apache/struts2/dispatcher/Dispatcher.java
---
@@ -88,7 +88,7 @@
*/
public static final String
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts/pull/151#discussion_r129282110
--- Diff: core/src/main/java/org/apache/struts2/dispatcher/Dispatcher.java
---
@@ -88,7 +88,7 @@
*/
public static final String
Github user sdutry commented on a diff in the pull request:
https://github.com/apache/struts/pull/151#discussion_r129283402
--- Diff: core/src/main/java/org/apache/struts2/dispatcher/Dispatcher.java
---
@@ -88,7 +88,7 @@
*/
public static final String
Github user sdutry commented on the issue:
https://github.com/apache/struts/pull/151
@lukaszlenart
I added 2 simple tests.
- one containing all the special allowed characters
- another one containing a single not-allowed character
Please feel free to tell me what
Github user sdutry commented on the issue:
https://github.com/apache/struts/pull/151
@lukaszlenart
Am i allowed to merge this or is there more work/checks that needs to
happen first?
---
If your project is set up for it, you can reply to this email and have your
reply appear on
GitHub user sdutry opened a pull request:
https://github.com/apache/struts/pull/152
WW-4826 upgrade to ASM version 5.2
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sdutry/struts WW-4826
Alternatively you can review and apply
GitHub user sdutry opened a pull request:
https://github.com/apache/struts/pull/156
WW-4834 Improve RegEx used to validate URLs
changed the regex according to
https://github.com/apache/struts/commit/8df5a897f61f3ef45c36fdd9275e9ae4516c#commitcomment-23414493
Can anyone
GitHub user sdutry opened a pull request:
https://github.com/apache/struts/pull/157
WW-4834 fixed faulty regex
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sdutry/struts WW-4834
Alternatively you can review and apply these
Github user sdutry commented on the issue:
https://github.com/apache/struts/pull/157
Ok,
I will wait.
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes
Github user sdutry commented on the issue:
https://github.com/apache/struts/pull/157
> After the commit above, any IP with 3 digits in the last octet will not
pass validation
You are right, i forgot the grouping (meaning the or statements mean
something completely differ
Github user sdutry commented on the issue:
https://github.com/apache/struts/pull/157
@lukaszlenart
Sorry for breaking it in the first place. That wasn't my intention.
Do you want me to merge this now, or am i still overlooking stuff?
---
If your project is set up f
GitHub user sdutry opened a pull request:
https://github.com/apache/struts/pull/159
WW-4834 Improve RegEx used to validate URLs
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sdutry/struts WW-4834-support-2-3
Alternatively you
Github user sdutry commented on the issue:
https://github.com/apache/struts/pull/159
see #157 and #156
---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or
Github user sdutry commented on the issue:
https://github.com/apache/struts/pull/159
@lukaszlenart
i copied the regex after the changes from #157 .
So it should be the same. Feel free to check.
---
If your project is set up for it, you can reply to this email and have your
GitHub user sdutry opened a pull request:
https://github.com/apache/struts/pull/166
WW-4851 Upgrade to Log4j2 2.9.0
You can merge this pull request into a Git repository by running:
$ git pull https://github.com/sdutry/struts WW-4851
Alternatively you can review and apply
52 matches
Mail list logo
