Closed Bug 783386 Opened 12 years ago Closed 12 years ago

Removing PATH_INFO prevents Bugzilla from working correctly with IIS

Categories

(Bugzilla :: Bugzilla-General, defect)

4.3.2
defect
Not set
critical

Tracking

()

RESOLVED FIXED
Bugzilla 4.4

People

(Reporter: LpSolit, Assigned: LpSolit)

References

()

Details

(Keywords: regression)

Attachments

(1 file)

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+
Summary: Removing PATH_INFO prevents Bugzilla to work correctly with IIS → Removing PATH_INFO prevents Bugzilla from working correctly with IIS
Attached patch patch, v1Splinter Review
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+
Flags: approval+
Committing to: bzr+ssh://lpsolit%40gmail.com@bzr.mozilla.org/bugzilla/trunk/
modified Bugzilla/CGI.pm
Committed revision 8347.
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
(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.

Attachment

General

Created:
Updated:
Size: