Opened on 09/10/2014 at 02:24:09 PM

Closed on 09/22/2014 at 05:42:06 PM

Last modified on 09/22/2014 at 05:55:28 PM

#1355 closed change (fixed)

avoid adblockplus.org to change its URL to platform-specific path

Reported by: philll Assignee: trev
Priority: P2 Milestone:
Module: Infrastructure Keywords:
Cc: till, trev Blocked By:
Blocking: Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working: no
Review URL(s):

http://codereview.adblockplus.org/5455756203655168

Description

Background

Journalists and other people linking to adblockplus.org very often don't recognize the platform-specific redirect and thus link to the URL matching their browser platform instead of the generic URL. Users opening this link with another browser are then confused about the grey download link. To avoid this to happen without breaking the possibility of specifically linking to a platform-specific version of the website, we want to have internal redirects instead of redirecting the user himself.

What to change

Avoid adblockplus.org to change its URL to a platform-specific path in the user's browser, while keeping the possibility of specifically linking to a platform-specific version of the website.

Attachments (0)

Change History (7)

comment:1 Changed on 09/10/2014 at 02:51:03 PM by trev

  • Owner set to trev

comment:2 Changed on 09/10/2014 at 03:03:13 PM by trev

  • Review URL(s) modified (diff)
  • Status changed from new to reviewing

I decided to push this change already because the approach has been discussed and Felix is currently on vacation. This still needs to be reviewed however.

comment:3 Changed on 09/10/2014 at 03:06:49 PM by trev

I had to implement redirects for language-independent URLs like https://adblockplus.org/chrome - selecting a different browser wouldn't work correctly otherwise.

There is also an effect that might not be desirable: https://adblockplus.org/bg won't redirect either. This means however that browser switching links point to https://adblockplus.org/chrome for example and ignore the selected language (Bulgarian in this case). So maybe we want to keep the redirect for https://adblockplus.org/bg. Alternatively, we can always redirect https://adblockplus.org/bg to https://adblockplus.org/bg/.

Last edited on 09/10/2014 at 03:09:01 PM by trev

comment:4 Changed on 09/10/2014 at 03:23:03 PM by trev

Alternatively, we can always redirect ​https://adblockplus.org/bg to ​https://adblockplus.org/bg/.

Implemented this, not having ambiguous URLs here seems to be a good idea anyway.

comment:5 Changed on 09/22/2014 at 05:42:06 PM by trev

  • Resolution set to fixed
  • Status changed from reviewing to closed

Got LGTM in the review. The changes themselves have been applied a while ago.

comment:6 Changed on 09/22/2014 at 05:48:39 PM by till

This fix doesn't solve the problem for existing links (e.g. here: http://www.huffingtonpost.com/2014/08/15/pop-up-ads-sorry-ethan-zuckerman_n_5681439.html).

comment:7 Changed on 09/22/2014 at 05:55:28 PM by trev

Yes, as discussed. We cannot do this without breaking the platform switching mechanism.

Add Comment

Modify Ticket

Change Properties
Action
as closed .
The resolution will be deleted. Next status will be 'reopened'.
to The owner will be changed from trev.
 
Note: See TracTickets for help on using tickets.