#1488226 closed Bugs (fixed)

invalid compose id

Reported by: corbosman Owned by:
Priority: 5 Milestone: 0.8-rc
Component: Core functionality Version: 0.7-beta
Severity: major Keywords:
Cc:

Description (last modified by alec)

I just upgraded a test server to 0.7 from 0.5 and am seeing the following error in my logs when i try to recall specific Draft msgs into compose..

Nov 30 10:51:40 roundcube7.xs4all.net roundcube: PHP Error: Invalid compose ID in /data/WWW/release-0.7/program/steps/mail/compose.inc on line 49 (GET /?_task=mail&_id=13045588924ed5fc2761713&_action=compose)

Change History (14)

comment:1 Changed 19 months ago by alec

  • Milestone changed from later to 0.7-stable

Works for me. Please, test with disabled plugins. Are you using two-DB-servers configuration or sth? WHat PHP version.

comment:2 Changed 19 months ago by corbosman

I am using a single DB server config. No read-only configured.

PHP 5.2.6-1+lenny13 with Suhosin-Patch 0.9.6.2 (cli) (built: Jul 1 2011 17:03:36)

I'll see if I can reproduce this on a dev server without plugins. Another one of those "doesnt happen often" problems, but with enough users you hit the edge cases.. few hundred times a day.

Cor

comment:3 Changed 19 months ago by dennylin93

I can also confirm this problem, but I do not know how to reproduce it. Using Roundcube 0.6 and PHP 5.3.8. Single MySQL database as backend.

comment:4 in reply to: ↑ description Changed 18 months ago by iezzip

I've encountered the same error in RC 0.7 on Debian Lenny under PHP 5.3.8 with Suhosin-patch 0.9.10. It does not to be a RC core problem, it looks like it's just a problem with the forwardattachment plugin from http://code.google.com/p/myroundcube/ (current SVN rev. 42).

The error occurrs once you try to select multiple messages from the message list (e.g. in INBOX) and then select from the top menu icon "forward - as attachment".

Currently, the forwardattachment plugin is buggy and is not able to forward multiple messages as attachment. Just disable the plugin as the single message forwarding (as attachment) is now integrated into RC. Selecting multiple messages was planned for RC 0.7, I thought. But it might make it into 0.8, let's hope.

comment:5 Changed 18 months ago by corbosman

We dont use that plugin.

comment:6 Changed 18 months ago by dennylin93

I don't use it as well.

comment:7 Changed 17 months ago by alec

How about now (0.7.1)?

comment:8 Changed 16 months ago by dennylin93

I haven't tried 0.7.1 yet.

comment:9 Changed 15 months ago by alec

Any more info? Is it cross-browser issue? Does it happen for all drafts or some specific?

comment:10 Changed 15 months ago by dennylin93

I am still seeing this error with 0.7.2, but I have no idea how to reproduce it.

comment:11 Changed 15 months ago by corbosman

grep 'Invalid compose ID' err-roundcube.log
Apr 8 09:13:08 roundcube9.xs4all.net roundcube: PHP Error: Invalid compose ID in /data/WWW/release-0.7/program/steps/mail/compose.inc on line 49 (GET /?_task=mail&_id=16602421844f813429546e3&_action=compose)

I still see them too, but not enough to even start to guess where it comes from. I cant reproduce it myself. Maybe we can add some debugging code?

comment:12 Changed 15 months ago by alec

I observed it once. When composing a message for long time, for some reason (no Internet connection or sth) I was logged off. Then after logging in Roundcube redirected me to compose with _id set in URL. Because the session was destroyed (on log out) the id wasn't valid anymore.
If this is the case we could:

  1. just not log the error or
  2. unset _id argument on login.

comment:13 Changed 14 months ago by alec

  • Description modified (diff)

This error logging was added as an infinite loop prevention for #1487028. Maybe we could get rid of it and additionaly instead of displaying error page do a redirect to ?_task=mail.

comment:14 Changed 14 months ago by alec

  • Resolution set to fixed
  • Status changed from new to closed

Re-login issue fixed in [d2191c61]. I hope this will limit the number of such errors in log. Please, reopen if needed.

Note: See TracTickets for help on using tickets.