Skip to content

Tolerate a leading / or ~/ on conventional routes #2765

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
Closed
jbagga opened this issue Jan 10, 2018 · 0 comments
Closed

Tolerate a leading / or ~/ on conventional routes #2765

jbagga opened this issue Jan 10, 2018 · 0 comments
Assignees
@jbagga
Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates Done This issue has been fixed
Milestone
2.1.0-preview1

Comments

@jbagga
Copy link
Contributor

jbagga commented Jan 10, 2018

From @rynowak on September 9, 2017 0:5

We throw an exception today if you do this, but we allow it in attribute routes. We should allow this.

When we generate a url using a route like "~/foo" the resulting url should be "/foo" not "~/foo"

Copied from original issue: aspnet/Routing#441
@jbagga jbagga self-assigned this Jan 10, 2018
@jbagga jbagga mentioned this issue Jan 10, 2018
Closed
@jbagga jbagga added this to the 2.1.0-preview1 milestone Jan 10, 2018
@jbagga jbagga mentioned this issue Jan 10, 2018
Merged
@jbagga jbagga added 2 - Working Done This issue has been fixed and removed 1 - Ready labels Jan 10, 2018
@jbagga jbagga closed this as completed Jan 11, 2018
@Eilon Eilon added area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates and removed repo:Routing labels Nov 7, 2018
ryanbrandenburg pushed a commit that referenced this issue Nov 27, 2018
@mikaelm12
Update Java client build scripts for Maven publishing (#2765)
5843b54
@ghost ghost locked as resolved and limited conversation to collaborators Dec 4, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees

@jbagga jbagga

Labels
area-mvc Includes: MVC, Actions and Controllers, Localization, CORS, most templates Done This issue has been fixed
Projects
None yet
Milestone
2.1.0-preview1
Development

No branches or pull requests
2 participants
@Eilon @jbagga