Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Bug] Paths.sanitizeRequestMappingPattern fix #2177

Closed
nobe0716 opened this issue Jan 9, 2018 · 0 comments · Fixed by #2178
Closed

[Bug] Paths.sanitizeRequestMappingPattern fix #2177

nobe0716 opened this issue Jan 9, 2018 · 0 comments · Fixed by #2178
Labels
Milestone

Comments

@nobe0716
Copy link
Contributor

nobe0716 commented Jan 9, 2018

Paths.sanitizeRequestMappingPattern works wrong with multi path variable between slashes
I copied java from source and changed regex to "\{([^}]+?):([^/{}]|\{[\d,]+\})+\}".

  def "should sanitize request mapping endpoints"() {
    expect:
    Paths.sanitizeRequestMappingPattern(mappingPattern) == expected

    where:
    mappingPattern                                  | expected
    "/{businessId:\\d{3}}:{productId:\\D{3}\\d{3}}" | "/{businessId}:{productId}"
  }
@nobe0716 nobe0716 changed the title Paths.sanitizeRequestMappingPattern bug report [Bug] Paths.sanitizeRequestMappingPattern fix Jan 9, 2018
dilipkrish added a commit that referenced this issue Jan 14, 2018
@ghost ghost removed the current label Jan 14, 2018
@dilipkrish dilipkrish added this to the 2.8.0 milestone Jan 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants