Removing PATH_INFO prevents Bugzilla from working correctly with IIS

RESOLVED FIXED in Bugzilla 4.4

Status

()

Bugzilla
Bugzilla-General
--
critical
RESOLVED FIXED
6 years ago
3 years ago

People

(Reporter: Frédéric Buclin, Assigned: Frédéric Buclin)

Tracking

({regression})

4.3.2
Bugzilla 4.4
regression
Bug Flags:
approval +
blocking4.4 +

Details

(URL)

Attachments

(1 attachment)

(Assignee)

Description

6 years ago
In bug 243764, I removed PATH_INFO as this was useless to Bugzilla, and could cause some problems when displaying pages, see the discussion there. But by doing this, Bugzilla no longer works with IIS, because IIS includes the full path to the CGI script in PATH_INFO, and so Bugzilla::CGI redirects the user to the www root dir, i.e. outside Bugzilla.

So we have two solutions at this point: either backing out bug 243764, or extracting the real PATH_INFO from what IIS sends us.
Flags: blocking4.4+
(Assignee)

Updated

6 years ago
Summary: Removing PATH_INFO prevents Bugzilla to work correctly with IIS → Removing PATH_INFO prevents Bugzilla from working correctly with IIS
(Assignee)

Comment 1

6 years ago
Created attachment 652632 [details] [diff] [review]
patch, v1

Tested with both Apache 2.2.22 and IIS 7.5.
Attachment #652632 - Flags: review?(dkl)
Comment on attachment 652632 [details] [diff] [review]
patch, v1

Review of attachment 652632 [details] [diff] [review]:
-----------------------------------------------------------------

Code looks correct and works for Apache. Cannot test on IIS so take your word that it that it works there. r=dkl
Attachment #652632 - Flags: review?(dkl) → review+
(Assignee)

Updated

6 years ago
Flags: approval+
(Assignee)

Comment 3

6 years ago
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified Bugzilla/CGI.pm
Committed revision 8347.
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
(Assignee)

Comment 4

3 years ago
(In reply to Frédéric Buclin from comment #0)
> IIS includes the full path to the CGI script in PATH_INFO

This problem has been fixed in CGI 4.09, see https://github.com/leejo/CGI.pm/issues/152.
You need to log in before you can comment on or make changes to this bug.