Opened 3 years ago

Closed 3 years ago

#20311 closed enhancement (fixed)

Remove redirects and update error page

Reported by: karsten Owned by: karsten
Priority: Medium Milestone: Metrics 1.0.0
Component: Metrics/Website Version:
Severity: Normal Keywords:
Cc: iwakeh Actual Points:
Parent ID: Points:
Reviewer: Sponsor:


We briefly discussed this at the metrics team meeting yesterday. There are still some quite old redirects in place on Tor Metrics. We concluded that we should turn them into "manual redirects" where we only display the link rather than redirecting users automatically, forcing them to notice and update their bookmarks accordingly. However, I read today that using 301 status codes is just fine for redirecting.

Now I wonder if the manual redirect idea might be a step backwards, because then there will be "content" again for search engines to index, which seems wrong. Maybe we can simply remove these redirects after 9 months. Users should notice that the URL has changed, shouldn't they?

While looking at the code I also found that our error page contains an outdated sitemap with many links that would need to be redirected. We should simply remove those links.

I'll attach a branch in a minute once I have a ticket number.

Child Tickets

Change History (5)

comment:2 Changed 3 years ago by iwakeh

Yes, that seems fine to remove the links. After all, users nowadays know that links can change.

Regarding the code I'd rather remove We have the git history that should be sufficient.

comment:3 Changed 3 years ago by karsten

Owner: set to karsten
Status: needs_reviewassigned

Heh, I guess I made that argument about Git history in other contexts, too. Didn't come to mind here, but I think you're right. Will remove that servlet later today. Thanks for the quick response!

comment:4 Changed 3 years ago by iwakeh

Milestone: Metrics 1.0.0

Added the milestone.

comment:5 Changed 3 years ago by karsten

Resolution: fixed
Status: assignedclosed

Merged, closing!

Note: See TracTickets for help on using tickets.