VertNet / usagestats

Codebase for the Usage Stat Reports generator
0 stars 0 forks source link

Trailing slash in URL #6

Open jotegui opened 8 years ago

jotegui commented 8 years ago

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 commented 8 years ago

Anything that helps people to not be confused is good.

On Fri, Jun 3, 2016 at 10:00 AM, Javier Otegui notifications@github.com 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 https://github.com/VertNet/usagestats/issues/6, or mute the thread https://github.com/notifications/unsubscribe/AAcP632HUNZeQ2GVIFh5dM6xqjuxGYaPks5qICVXgaJpZM4ItgfH .

dbloom commented 8 years ago

Helping people is good.