Error in RequestTemplate#uri when there're both query and fragment (#2367)

Author: gromspys

core/src/main/java/feign/RequestTemplate.java

@@ -457,6 +457,12 @@ public RequestTemplate uri(String uri, boolean append) {
       uri = "/" + uri;
     }
 
+    int fragmentIndex = uri.indexOf('#');
+    if (fragmentIndex > -1) {
+      fragment = uri.substring(fragmentIndex);
+      uri = uri.substring(0, fragmentIndex);
+    }
+
     /*
      * templates may provide query parameters. since we want to manage those explicity, we will need
      * to extract those out, leaving the uriTemplate with only the path to deal with.
@@ -472,12 +478,6 @@ public RequestTemplate uri(String uri, boolean append) {
       uri = uri.substring(0, queryMatcher.start());
     }
 
-    int fragmentIndex = uri.indexOf('#');
-    if (fragmentIndex > -1) {
-      fragment = uri.substring(fragmentIndex);
-      uri = uri.substring(0, fragmentIndex);
-    }
-
     /* replace the uri template */
     if (append && this.uriTemplate != null) {
       this.uriTemplate = UriTemplate.append(this.uriTemplate, uri);

core/src/test/java/feign/RequestTemplateTest.java

@@ -462,6 +462,16 @@ void fragmentShouldNotBeEncodedInTarget() {
     assertThat(template.url()).isEqualTo("https://example.com/path?key1=value1#fragment");
   }
 
+  @Test
+  void fragmentShouldBeExtractedWhenQueryParamsExist() {
+    RequestTemplate template =
+            new RequestTemplate().method(HttpMethod.GET).uri("/path?query=queryValue#fragment",
+                    true);
+
+    assertThat(template.url()).isEqualTo("/path?query=queryValue#fragment");
+    assertThat(template.queryLine()).isEqualTo("?query=queryValue");
+  }
+
   @Test
   void urlEncodingRemainsInPlace() {
     RequestTemplate template = new RequestTemplate().method(HttpMethod.GET)