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)
Tracking
()
RESOLVED
FIXED
Bugzilla 4.4
People
(Reporter: LpSolit, Assigned: LpSolit)
References
()
Details
(Keywords: regression)
Attachments
(1 file)
|
1.01 KB,
patch
|
dkl
:
review+
|
Details | Diff | Splinter Review |
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•12 years ago
|
Summary: Removing PATH_INFO prevents Bugzilla to work correctly with IIS → Removing PATH_INFO prevents Bugzilla from working correctly with IIS
|
|
Assignee | |
Updated•12 years ago
|
|
|
Assignee | |
Comment 1•12 years ago
|
||
Tested with both Apache 2.2.22 and IIS 7.5.
Attachment #652632 -
Flags: review?(dkl)
|
||
Comment 2•12 years ago
|
||
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•12 years ago
|
Flags: approval+
|
|
Assignee | |
Comment 3•12 years ago
|
||
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
|
|
Assignee | |
Comment 4•10 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.
Description
•