Opened on 10/08/2015 at 02:29:44 PM

Closed on 11/02/2015 at 12:40:01 PM

#3180 closed defect (fixed)

Work around outlook URL rewrites

Reported by: matze Assignee: matze
Priority: P2 Milestone:
Module: Infrastructure Keywords:
Cc: fred, oleksandr, saroyanm Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: yes Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

https://codereview.adblockplus.org/29329011

Description

When receiving an Edge verification mail, for example, Outlook 10 seems to rewrite the target URL, violating compliance with RFC 2396:

https://adblockplus.org/verifyEmail?email=example%2540example.com&signature=...&product=edge&lang=en

The above should actually appear as follows:

https://adblockplus.org/verifyEmail?email=example%40example.com&signature=...&product=edge&lang=en

Note the %40 (@) being re-written as %2540 (%40).

Todo

Find a workaround, e.g. an Nginx rewrite, for this case in particular, and encourage our developers to avoid placing plain e-mail addresses in query strings.

Attachments (0)

Change History (3)

comment:1 Changed on 10/09/2015 at 01:16:33 PM by matze

  • Owner set to matze

comment:2 Changed on 10/09/2015 at 01:16:44 PM by matze

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

comment:3 Changed on 11/02/2015 at 12:40:01 PM by matze

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

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 matze.
 
Note: See TracTickets for help on using tickets.