Make WordPress Core

Opened 13 years ago

Last modified 5 years ago

#17450 reopened defect (bug)

Almost pretty permalinks with non-ASCII characters do not work in Apache

Reported by: peaceablewhale's profile peaceablewhale Owned by: westi's profile westi
Milestone: Priority: normal
Severity: normal Version: 3.1
Component: Permalinks Keywords: needs-unit-tests needs-refresh
Focuses: Cc:

Description

Almost pretty permalinks (using PATH_INFO) with non-ASCII characters do not work in Apache; a not found error is returned. The same permalink works in IIS when UTF-8 is used for server variables. It also works when mod_rewrite is used.

Attachments (2)

17450.diff (975 bytes) - added by solarissmoke 13 years ago.
17450.patch (770 bytes) - added by peaceablewhale 13 years ago.

Download all attachments as: .zip

Change History (16)

@solarissmoke
13 years ago

#1 @solarissmoke
13 years ago

  • Keywords has-patch needs-testing added

The problem is not just with Apache. Here's the issue: PATH_INFO is (according to the CGI spec) supplied url-decoded, but REQUEST_URI us url-encoded. !Wordpress does string comparisons on these two, which is fine when there are only ascii characters in the URL, but fails when there are others.

Here is a patch that works for me. I imagine it will need some testing to make sure the behaviour is consistent in all environments.

#2 @peaceablewhale
13 years ago

Patch breaks almost pretty permalinks in IIS. Trying to fix it.

#3 @peaceablewhale
13 years ago

Revised patch. Workaround now applies to Apache only.

#4 @peaceablewhale
13 years ago

Unfortunately, encoding PATH_INFO does not work either. I have reversed the patch logic so that we now decode REQUEST_URI instead. Verified effective in both IIS and Apache.

#5 @peaceablewhale
13 years ago

  • Version changed from 3.1.2 to 3.2

#6 @peaceablewhale
13 years ago

  • Owner set to westi
  • Status changed from new to reviewing

westi, I guess you may be the appropriate person to review this ticket, as I noticed your experience in resolving tickets related to servers~

#7 @SergeyBiryukov
13 years ago

  • Milestone Awaiting Review deleted
  • Resolution set to duplicate
  • Status changed from reviewing to closed

This is caused by [15960] and can be fixed by the latest patches in #10249.

#8 @SergeyBiryukov
13 years ago

  • Milestone set to 3.3
  • Resolution duplicate deleted
  • Status changed from closed to reopened
  • Version changed from 3.2 to 3.1

Not exactly a duplicate of the original #10249 bug, just related. 3.1 is the earliest affected version.

Still, 10249.4.patch fixes that.

Version 0, edited 13 years ago by SergeyBiryukov (next)

#9 @SergeyBiryukov
12 years ago

  • Keywords needs-unit-tests added

#10 @SergeyBiryukov
12 years ago

  • Milestone changed from 3.3 to Future Release

#11 @peaceablewhale
12 years ago

How may unit test cases be made?

#12 @SergeyBiryukov
12 years ago

  • Milestone changed from Future Release to 3.4

#13 @nacin
12 years ago

  • Keywords 3.5-early added
  • Milestone changed from 3.4 to Future Release

Keeping on the same path as #10249.

#14 @chriscct7
9 years ago

  • Keywords needs-refresh added; has-patch needs-testing 3.5-early removed
  • Severity changed from major to normal

This and #10249 could be fixed and tested together probably considering comment:8

Note: See TracTickets for help on using tickets.