Opened 2 years ago
Closed 2 years ago
#1487892 closed Bugs (fixed)
Addressbook : "Mail to" failed when selected too many people
| Reported by: | bn8net | Owned by: | |
|---|---|---|---|
| Priority: | 5 | Milestone: | 0.6-beta |
| Component: | Addressbook | Version: | 0.5.2 |
| Severity: | normal | Keywords: | addressbook |
| Cc: |
Description
Hello,
In address book, if I select too many people and click on "Send message to" button, I have this error :
Server error ! (Request-URI Too Large)
Address book used for this action an HTTP GET request which is limited (not in RFC but in browser or by server). It could be better to use an HTTP POST request. I wrote a little patch for this:
Index: program/js/app.js
===================================================================
--- program/js/app.js (révision 4710)
+++ program/js/app.js (copie de travail)
@@ -840,7 +840,7 @@
}
if (a_cids.length)
- this.http_request('mailto', '_cid='+urlencode(a_cids.join(','))+'&_source='+urlencode(this.env.source), true);
+ this.http_post('mailto', { _cid: a_cids.join(','), _source: this.env.source, _task: 'addressbook' } , true);
break;
}
Index: program/steps/addressbook/mailto.inc
===================================================================
--- program/steps/addressbook/mailto.inc (révision 4710)
+++ program/steps/addressbook/mailto.inc (copie de travail)
@@ -19,7 +19,7 @@
*/
-$cid = get_input_value('_cid', RCUBE_INPUT_GET);
+$cid = get_input_value('_cid', RCUBE_INPUT_POST);
$recipients = null;
$mailto = array();
I tested this patch on trunk and 0.5.2 versions.
Change History (1)
comment:1 Changed 2 years ago by alec
- Milestone changed from later to 0.6-beta
- Resolution set to fixed
- Status changed from new to closed
Note: See
TracTickets for help on using
tickets.

Fixed in [cf58ce85].