[GH-ISSUE #343] amavis extension: internal error when deleting messages on last page of a multi-page quarantine #326

New issue

Closed

opened 2026-02-27 11:11:12 +03:00 by kerem · 3 comments

kerem commented

2026-02-27 11:11:12 +03:00

Owner

Originally created by @tonioo on GitHub (Dec 4, 2013).
Original GitHub issue: https://github.com/modoboa/modoboa/issues/343

Originally assigned to: @tonioo on GitHub.

Originally created by Louis-Dominique Dubeau on 2012-10-26T12:31:31Z

Modoboa version 0.9.3

(However it also happened with 0.9.2. Forgot to report it. Sorry.)

This is easily reproducible on my installation:

Wait until the quarantine is big enough to require more than one page to display.
Go to the last page of the quarantine.
Select some messages to delete and delete them.
Modoboa will display an "Internal error" message. (A red notification at the top of the page.)

I considered that maybe the problem is that if all the messages on the last page (let's call it page N) are deleted Modoboa still tries to display page N instead of page N-1 and fails because page N no longer exists since all the messages that would be on it have been deleted. However, the error happens even if only a subset of messages on page N are deleted and thus still exists after the deletion operation.

Originally created by @tonioo on GitHub (Dec 4, 2013). Original GitHub issue: https://github.com/modoboa/modoboa/issues/343 Originally assigned to: @tonioo on GitHub. **Originally created by Louis-Dominique Dubeau on 2012-10-26T12:31:31Z** Modoboa version 0.9.3 (However it also happened with 0.9.2. Forgot to report it. Sorry.) This is easily reproducible on my installation: 1. Wait until the quarantine is big enough to require more than one page to display. 2. Go to the last page of the quarantine. 3. Select some messages to delete and delete them. 4. Modoboa will display an "Internal error" message. (A red notification at the top of the page.) I considered that maybe the problem is that if all the messages on the last page (let's call it page N) are deleted Modoboa still tries to display page N instead of page N-1 and fails because page N no longer exists since all the messages that would be on it have been deleted. However, the error happens even if only a subset of messages on page N are deleted and thus _still_ exists after the deletion operation.

kerem

2026-02-27 11:11:12 +03:00

closed this issue
added the
bug
label

kerem commented

2026-02-27 11:11:13 +03:00

Author

Owner

@tonioo commented on GitHub (Dec 4, 2013):

Posted by Antoine Nguyen on 2012-10-26T19:25:24Z

Hi Louis-Dominique,

can you tell me which SGBD you're using?

@tonioo commented on GitHub (Dec 4, 2013): **Posted by Antoine Nguyen on 2012-10-26T19:25:24Z** Hi Louis-Dominique, can you tell me which SGBD you're using?

kerem commented

2026-02-27 11:11:13 +03:00

Author

Owner

@tonioo commented on GitHub (Dec 4, 2013):

Posted by Antoine Nguyen on 2012-10-26T20:01:27Z

Applied in changeset commit:de9065a80b9a.

@tonioo commented on GitHub (Dec 4, 2013): **Posted by Antoine Nguyen on 2012-10-26T20:01:27Z** Applied in changeset commit:de9065a80b9a.

kerem commented

2026-02-27 11:11:13 +03:00

Author

Owner

@tonioo commented on GitHub (Dec 4, 2013):

Posted by Antoine Nguyen on 2012-10-26T20:02:38Z

Actually, the issue was due to a wrong URL manipulation (the '#listing/' part was removed after the operation).

@tonioo commented on GitHub (Dec 4, 2013): **Posted by Antoine Nguyen on 2012-10-26T20:02:38Z** Actually, the issue was due to a wrong URL manipulation (the '#listing/' part was removed after the operation).