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

Trailing slash in URL #6

Open
jotegui opened this issue Jun 3, 2016 · 2 comments
Open

Trailing slash in URL #6

jotegui opened this issue Jun 3, 2016 · 2 comments

Comments

@jotegui
Copy link
Member

jotegui commented Jun 3, 2016

I see in the logs that many people are trying to access the reports without the trailing slash, which throws a 404 error. The URL in the email notification is built with the slash, and all inner links work fine, but if one hard-types the URL, it is easy to forget the final slash. From the building perspective, the slash is important for the proper functioning of the JSON and TXT versions of the reports.

Still, maybe it would be better to enable redirect of non-slash URLs to the proper ones?

@tucotuco
Copy link
Member

tucotuco commented Jun 3, 2016

Anything that helps people to not be confused is good.

On Fri, Jun 3, 2016 at 10:00 AM, Javier Otegui [email protected]
wrote:

I see in the logs that many people are trying to access the reports
without the trailing slash, which throws a 404 error. The URL in the email
notification is built with the slash, and all inner links work fine, but if
one hard-types the URL, it is easy to forget the final slash. From the
building perspective, the slash is important for the proper functioning of
the JSON and TXT versions of the reports.

Still, maybe it would be better to enable redirect of non-slash URLs to
the proper ones?


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#6, or mute the thread
https://github.com/notifications/unsubscribe/AAcP632HUNZeQ2GVIFh5dM6xqjuxGYaPks5qICVXgaJpZM4ItgfH
.

@dbloom
Copy link
Member

dbloom commented Jun 3, 2016

Helping people is good.

@jotegui jotegui removed the question label Jun 8, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

3 participants