Revert "Fixing trailing characters treated as part of URI error (#3938)."

Daniel Gultsch created

This reverts commit 8d45cc5827f9fb3f6535749cc7d5c624fc08f5b5.

Change summary

src/main/java/eu/siacs/conversations/utils/Patterns.java | 12 ++-------
1 file changed, 3 insertions(+), 9 deletions(-)

Detailed changes

src/main/java/eu/siacs/conversations/utils/Patterns.java 🔗

@@ -337,15 +337,9 @@ public class Patterns {
             + "\\,\\;\\?\\&\\=]|(?:\\%[a-fA-F0-9]{2})){1,64}(?:\\:(?:[a-zA-Z0-9\\$\\-\\_"
             + "\\.\\+\\!\\*\\'\\(\\)\\,\\;\\?\\&\\=]|(?:\\%[a-fA-F0-9]{2})){1,25})?\\@";
     private static final String PORT_NUMBER = "\\:\\d{1,5}";
-    private static final String PATH_AND_QUERY_CHARS_WITHOUT_SLASH =
-            "\\;\\?\\:\\@\\&\\=\\#\\~" // plus optional query params
-            + "\\-\\.\\+\\!\\*\\'\\(\\)\\,\\_\\$";
     private static final String PATH_AND_QUERY = "\\/(?:(?:[" + LABEL_CHAR
-            + PATH_AND_QUERY_CHARS_WITHOUT_SLASH
-            + "]+[^" + PATH_AND_QUERY_CHARS_WITHOUT_SLASH + "\\s]" // path and query chars must not be trailing
-            + "|\\/" // trailing slashes are fine
-            + ")|(?:\\%[a-fA-F0-9]{2}))*";
-
+            + "\\;\\/\\?\\:\\@\\&\\=\\#\\~"  // plus optional query params
+            + "\\-\\.\\+\\!\\*\\'\\(\\)\\,\\_\\$])|(?:\\%[a-fA-F0-9]{2}))*";
     /**
      *  Regular expression pattern to match most part of RFC 3987
      *  Internationalized URLs, aka IRIs.
@@ -516,4 +510,4 @@ public class Patterns {
      * Do not create this static utility class.
      */
     private Patterns() {}
-}
+}