Closed
Bug 1258187
Opened 10 years ago
Closed 10 years ago
When using plackup as a proxy, Apache is unable to handle saved queries correctly
Categories
(Bugzilla :: Query/Bug List, defect)
Tracking
()
RESOLVED
FIXED
Bugzilla 6.0
People
(Reporter: LpSolit, Assigned: LpSolit)
References
Details
Attachments
(1 file)
|
647 bytes,
patch
|
dkl
:
review+
|
Details | Diff | Splinter Review |
Follow instructions from bug 1258157 to use plackup from Apache. $cgi->redirect_search_url redirects saved queries to the wrong URL (using the internal URL used by plackup, i.e. localhost:5000), because it's unaware of the proxy used by Apache. $cgi->redirect_to_urlbase prepends the urlbase to generate an absolute path, so we should use it.
Attachment #8732617 -
Flags: review?(dkl)
Comment 1•10 years ago
|
||
Comment on attachment 8732617 [details] [diff] [review]
bug0-14.diff
Review of attachment 8732617 [details] [diff] [review]:
-----------------------------------------------------------------
Fixes the issue. r=dkl
Attachment #8732617 -
Flags: review?(dkl) → review+
| Assignee | ||
Comment 2•10 years ago
|
||
To ssh://gitolite3@git.mozilla.org/bugzilla/bugzilla.git
da0347e..c7e8031 master -> master
Status: ASSIGNED → RESOLVED
Closed: 10 years ago
Resolution: --- → FIXED
| Assignee | ||
Comment 4•10 years ago
|
||
Do we want this fix for 5.0.3 too based on comments in bug 1063394?
Flags: approval5.0?
| Assignee | ||
Comment 6•10 years ago
|
||
There is much more to do to have a fully functional installation behind a reverse proxy, see e.g. bug 1261538. So there is no need to take this one for 5.0 as it wouldn't be a complete fix anyway.
Flags: approval5.0?
You need to log in
before you can comment on or make changes to this bug.
Description
•