crunchuser / prosody-modules

Automatically exported from code.google.com/p/prosody-modules
MIT License
0 stars 0 forks source link

[mod_mam_sql] 0.8.x version is not able to reconnect after connection is lost #28

Open GoogleCodeExporter opened 9 years ago

GoogleCodeExporter commented 9 years ago
I am running 0.8.x edition of this plugin (without shared connections), rev 
[f58a2c5ae6ef], together with Prosody 0.8.2, Lua 5.1 (Debian Wheezy).
(So I cannot decide whether this bug is still valid in 9.0 version)

Plugin is not able to handle accidental DB disconnect (e.g. Mysql server 
restart) - the rest of Prosody seems to be able to do so. Furthermore, it 
causes no messages are even delivered, because calling error() on line 101: 
function drops everything.

To reproduce
1. Run prosody
2. Send some message - sent & logged
3. Restart mysql server
4. Send another message - raises an error and does not deliver the message at 
all

Original issue reported on code.google.com by mar...@mplicka.cz on 27 Jun 2013 at 9:07

GoogleCodeExporter commented 9 years ago
Traceback:

Original comment by mar...@mplicka.cz on 30 Jun 2013 at 3:47

Attachments:

GoogleCodeExporter commented 9 years ago
Well, it seems that the error is raised in the next run when connection 
variable gets nil by the test_connection() function.

Original comment by mar...@mplicka.cz on 30 Jun 2013 at 4:42