Summary | Navigating to Mail Search from top menu error: URI Too Large |
Queue | IMP |
Queue Version | 6.1.3 |
Type | Bug |
State | Resolved |
Priority | 1. Low |
Owners | slusarz (at) horde (dot) org |
Requester | delrio (at) mie (dot) utoronto (dot) ca |
Created | 05/29/2013 (4431 days ago) |
Due | |
Updated | 08/20/2013 (4348 days ago) |
Assigned | 08/14/2013 (4354 days ago) |
Resolved | 08/20/2013 (4348 days ago) |
Github Issue Link | |
Github Pull Request | |
Milestone | |
Patch | No |
Assigned to Michael Slusarz
State ⇒ Resolved
Version ⇒ 6.1.3
commit bf8518dcb95d7fecbf25976ecced0d69a1d1113e
Author: Michael M Slusarz <slusarz@horde.org>
Date: Tue Aug 20 00:16:42 2013 -0600
Bug #12283: Fix URL generation when loading Search from topbarfrom a non-IMP page
imp/js/dimpbase.js | 5 +++++
1 files changed, 5 insertions(+), 0 deletions(-)
http://git.horde.org/horde-git/-/commit/bf8518dcb95d7fecbf25976ecced0d69a1d1113e
sends me to:
/horde/imp/dynamic.php?page=mailbox#search
Address:
/imp/basic.php?0=S&1=U&2=5&3=C&4=T&5=1&6=g&page=search&gsub=function+gsub(pattern%2Creplacement)%7Bvar+result.....
It can be duplicated on demo.horde.org (guest/guest). The resulting
URL in the demo site is just short of the 8190 apache limit (about
8000 chars) so no error is displayed in this server.
When called from the Mail application, the frame's address is
/imp/basic.php?page=search&mailbox=SU5CT1g&token=jfDILj-Hy-jX5PSb6jHUIQ6
line 255, the passed "data" parameter is SU5CT1g, because that's
what's passed to go() on line 3804. Since Hash.updated is expected
to be passed a traversable object, all String properties are added
to the hash, that includes all String methods.
sends me to:
/horde/imp/dynamic.php?page=mailbox#search
NO mailbox information (e.g. SU5CT1g) should appear in this URL:
$url = Horde::url('dynamic.php')
->add('page', 'mailbox')
->setAnchor('search');
So I can't determine a way how the 'tmp' variable on line 3804
contains anything other than 'search'.
State ⇒ Feedback
255, the passed "data" parameter is SU5CT1g, because that's what's
passed to go() on line 3804. Since Hash.updated is expected to be
passed a traversable object, all String properties are added to the
hash, that includes all String methods.
State ⇒ No Feedback
/imp/basic.php?page=search&0=S&1=U&2=5&3=C&4=T&5=1&6=g&gsub=function+gsub(pattern%2C+replacement)+%7B%0D%0A++++var+result+%3D+%22%22%2C+source+%3D+this%2C+match%3B%0D%0A++++replacement+%3D+prepareReplacement(replacement)%3B%0D%0A++++if+(Object.isString(pattern))......... and goes on and on with what seems to be javascript
code.
The HTML code for me looks like:
<div class="horde-drowdown-str"><a class="horde-mainnavi"
href="/horde/imp/basic.php?page=search">Search</a></div>
The URL is generated via: IMP_Basic_Search::url(), which is just:
Horde::url('basic.php')->add('page', 'search')
That's as simple as you can get. There is nothing being dynamically
added to basic.php base page URI except "page=search".
The javascript code appears to be from prototype.js. But I have no
idea how it is getting there.
see the requested URIs on the server side. The requests when clicking
Mail - Search go like this:
GET /static/fb92fc0212b4afa89b6a7c924da5c484.css
GET /static/639970d9474db9783e4535eafa252a5900b5df07.js
POST /services/ajax.php/imp/dynamicInit
/imp/basic.php?page=search&0=S&1=U&2=5&3=C&4=T&5=1&6=g&gsub=function+gsub(pattern%2C+replacement)+%7B%0D%0A++++var+result+%3D+%22%22%2C+source+%3D+this%2C+match%3B%0D%0A++++replacement+%3D+prepareReplacement(replacement)%3B%0D%0A++++if+(Object.isString(pattern))......... and goes on and on with what seems to be javascript
code.
Book, choosing Mail - Search from the top menu fails.
URI I see, and it is significantly smaller than 8000+ characters.
works fine.
But if you are in another application such as Calendar or Address
Book, choosing Mail - Search from the top menu fails. Similar to Bug
#12273but in that case the error was "error communicating with theserver" instead of "URI too long"
The URL should look like:
https://example.com/horde/imp/dynamic.php?page=mailbox#search
Priority ⇒ 1. Low
State ⇒ Unconfirmed
Patch ⇒ No
Milestone ⇒
Summary ⇒ Navigating to Mail Search from top menu error: URI Too Large
Type ⇒ Bug
Queue ⇒ IMP
Bug #12273Navigating to Mail Search from top menu from another application in
dynamic mode now produces the following error:
Request-URI Too Large
The server logs: "request failed: URI too long (longer than 8190)"