Skip to content

Commit d7561ef

Browse files
Merge pull request #556 from gabriel-farache/fix_urlpattermatcher (#557)
Fix UrlPatternMatcher not matching parameters in path containing "-" character Co-authored-by: Francisco Javier Tirado Sarti <[email protected]>
1 parent d8363d7 commit d7561ef

File tree

2 files changed

+3
-2
lines changed

2 files changed

+3
-2
lines changed

runtime/src/main/java/io/quarkiverse/openapi/generator/providers/UrlPatternMatcher.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,7 +64,7 @@
6464
*/
6565
public class UrlPatternMatcher {
6666
// Finds parameters in the URL pattern string.
67-
private static final String URL_PARAM_REGEX = "\\{(\\w*?)\\}";
67+
private static final String URL_PARAM_REGEX = "\\{(\\S*?)\\}";
6868

6969
// Replaces parameter names in the URL pattern string to match parameters in URLs.
7070
private static final String URL_PARAM_MATCH_REGEX = "\\([%\\\\w-.\\\\~!\\$&'\\\\(\\\\)\\\\*\\\\+,;=:\\\\[\\\\]@]+?\\)";

runtime/src/test/java/io/quarkiverse/openapi/generator/providers/UrlPatternMatcherTest.java

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,7 +31,8 @@ private static Stream<Arguments> providePathsThatMatch() {
3131
Arguments.of("/{id}/pets/{id2}", "/1/pets/2?q=1&q2=2"),
3232
Arguments.of("/{id}/{foo}/{id2}", "/1/2/3?q=1&q2=2"),
3333
Arguments.of("/{id}/{foo}/{id2}", "/1/2/3"),
34-
Arguments.of("/v2/pets/{id}", "/v2/pets/1"));
34+
Arguments.of("/v2/pets/{id}", "/v2/pets/1"),
35+
Arguments.of("/pets/{pet-id}/types/{type-id}", "/pets/1/types/2"));
3536
}
3637

3738
}

0 commit comments

Comments
 (0)