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

Default url parsing logic does not enforce trailing slash or not #8370

Closed
nkovacs opened this issue May 12, 2015 · 2 comments · Fixed by #12240
Closed

Default url parsing logic does not enforce trailing slash or not #8370

nkovacs opened this issue May 12, 2015 · 2 comments · Fixed by #12240
Assignees
@samdark
Labels
type:bug Bug
Milestone
2.0.10

Comments

@nkovacs
Copy link
Contributor

nkovacs commented May 12, 2015

If you have actionIndex in SiteController, site/index and site/index/ both work. But if you create a url rule, e.g.:

'foo/<action:\w+>' => 'site/<action>'

then only foo/index will work, because of #1807.
@cebe
Copy link
Member

cebe commented May 12, 2015

duplicate of #7670?

@nkovacs
Copy link
Contributor Author

nkovacs commented May 12, 2015

No. site/index/ should not work either, for the same reasons as #1807, but that would break BC.

@cebe cebe added this to the 2.0.x milestone May 12, 2015
@cebe cebe mentioned this issue Jun 7, 2015
Closed
@rob006 rob006 mentioned this issue Aug 21, 2016
Merged
3 tasks
@SilverFire SilverFire added the type:bug Bug label Aug 31, 2016
@SilverFire SilverFire modified the milestones: 2.0.10, 2.0.x Aug 31, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@samdark samdark

Labels
type:bug Bug
Projects
None yet
Milestone
2.0.10
Development

Successfully merging a pull request may close this issue.

Add UrlNormalizer for UrlRule and UrlManager rob006/yii2-dev
4 participants
@samdark @cebe @nkovacs @SilverFire