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

Fixed in [cf58ce85].

Note: See TracTickets for help on using tickets.