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.

Sign up for GitHub

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

Jump to bottom

feature: add a new option, decide if we remove the "/" at the end of the URI before route mathing #1765

Closed
membphis opened this issue Jun 24, 2020 · 0 comments · Fixed by #1766
Closed

feature: add a new option, decide if we remove the "/" at the end of the URI before route mathing #1765

membphis opened this issue Jun 24, 2020 · 0 comments · Fixed by #1766
Assignees
@membphis
Labels
enhancement New feature or request

Comments

@membphis
Copy link
Member

here is the discuss mail: https://lists.apache.org/thread.html/rd89e4e0a2d59e183afa00c8c1badf213ef45c0992f841046e129a965%40%3Cdev.apisix.apache.org%3E

add new option delete_uri_tail_slash in conf/config.yaml.

  delete_uri_tail_slash: true     # delete the '/' at the end of the URI
  router:
    http: 'radixtree_uri'         # radixtree_uri: match route by uri(base on radixtree)
                                  # radixtree_host_uri: match route by host + uri(base on radixtree)
    ssl: 'radixtree_sni'          # radixtree_sni: match route by SNI(base on radixtree)
@membphis membphis added the enhancement New feature or request label Jun 24, 2020
@membphis membphis self-assigned this Jun 24, 2020
@membphis membphis mentioned this issue Jun 24, 2020
Merged
membphis added a commit that referenced this issue Jul 29, 2020
@membphis
feature: add a new option, decide if we remove the "/" at the end of … (
f9a8a26 
#1766)

* feature: add a new option, decide if we remove the "/" at the end of the URI before route matching.

fix #1765
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@membphis membphis

Labels
enhancement New feature or request
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

feature: add a new option, decide if we remove the "/" at the end of …
1 participant
@membphis