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

Already on GitHub? Sign in to your account

uri_for修正 #7

Merged
merged 1 commit into from Oct 12, 2012

Conversation

Projects
None yet
2 participants
Contributor

ichirin2501 commented Oct 12, 2012

Plack::Builderのmountで、
'/admin' , '/admin/' どちらも '/admin' 扱いになり、最後の'/'が削られる。
Ark/Context.pm -> uri_for 内部で使用されている URI::WithBase のbaseは
最後の'/'までしか読み込まない。

http://localhost/admin => http://localhost/
http://localhost/admin/ => http://localhost/admin/
これらによって起きるバグを修正しました。
レビュー宜しくお願いします。

typester added a commit that referenced this pull request Oct 12, 2012

Merge pull request #7 from ichirin2501/master
Fixed a uri_for bug doesn't handle req->base correctly if base path end without trailing slash

@typester typester merged commit f50cc70 into typester:master Oct 12, 2012

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