Opened on 03/19/2015 at 10:04:45 PM

Closed on 03/20/2015 at 04:22:51 PM

#2181 closed change (fixed)

[cms] Test server should accept default page for subdirectories as well

Reported by: trev Assignee: trev
Priority: P3 Milestone:
Module: Sitescripts Keywords:
Cc: saroyanm Blocked By:
Blocking: #662 Platform: Unknown
Ready: yes Confidential: no
Tester: Verified working:
Review URL(s):

​http://codereview.adblockplus.org/5217703262420992/

Description

Background

Currently, if the default page is index only a request to / or /en/ will be internally redirected to /en/index. A request to /en/foo/ won't be redirected to /en/foo/index however. This only affects the test server, the real server relies on nginx functionality for this which will work for any directory.

What to change

Try appending defaultpage to any request, not just requests for the server root. Also do that when resolving links.

Attachments (0)

Change History (2)

comment:1 Changed on 03/19/2015 at 10:34:05 PM by trev

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

comment:2 Changed on 03/20/2015 at 04:22:51 PM by trev

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

Fixed: ​https://hg.adblockplus.org/cms/rev/ca5c183cff81

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