Tickets :: [#7350] Document that the session timeout draft code requires use_vfs

6.0.0-beta1

7/17/25

Summary	Document that the session timeout draft code requires use_vfs
Queue	IMP
Queue Version	HEAD
Type	Enhancement
State	Resolved
Priority	1. Low
Owners	slusarz (at) horde (dot) org
Requester	jan (at) horde (dot) org
Created	09/17/2008 (6147 days ago)
Due
Updated	01/12/2010 (5665 days ago)
Assigned	11/06/2008 (6097 days ago)
Resolved	12/03/2008 (6070 days ago)
Milestone
Patch	No

01/12/2010 11:55:15 PM	CVS Commit	Comment #6	Reply to this comment
Changes have been made in Git for this ticket: See ~~Ticket #7350~~. http://git.horde.org/diff.php/imp/docs/INSTALL?rt=horde-git&r1=d549da52517ff20cb889ba9eb953dd7499318754&r2=8e90726f32e91cf40b51f840e776c00eb0031738

12/03/2008 06:36:55 AM	Michael Slusarz	Version ⇒ HEAD

12/03/2008 06:36:34 AM	Michael Slusarz	Comment #5 Type ⇒ Enhancement State ⇒ Resolved Priority ⇒ 1. Low	Reply to this comment
This has been added to IMP 5 docs. (Saving drafts to the VFS backend was never backported to IMP 4.3, so this doesn't appear there).

11/06/2008 02:16:11 AM	Chuck Hagenbuch	Comment #4 Summary ⇒ Document that the session timeout draft code requires use_vfs State ⇒ Assigned Assigned to Michael Slusarz	Reply to this comment
Updating the ticket to focus on documenting the functionality and updating the description of use_vfs.

09/22/2008 04:40:50 AM	Michael Slusarz	Comment #3	Reply to this comment
I disagree. That code never worked for me, and definitely won't work if IMP is not the auth handler, if a custom login page is used, if maintenace tasks are run, etc. In other words, having a half-assed solution that requires no configuration but works terribly unreliably is absolutely no replacement for having a solution that works (lingering bugs aside) even though it requires additional configuration/resources.

09/22/2008 04:39:08 AM	Michael Slusarz	Comment #2	Reply to this comment
I think we should have kept the old timeout code for the case that vfs is not available, but we should at least document that vfs is required for that feature now. I disagree. That code never worked for me, and definitely won't work if IMP is not the auth handler, if a custom login page is used, if maintenace tasks are run, etc.

09/17/2008 02:34:59 PM	Jan Schneider	Comment #1 Priority ⇒ 2. Medium State ⇒ Unconfirmed Patch ⇒ No Milestone ⇒ Queue ⇒ IMP Summary ⇒ No drafts on session timeout without use_vfs Type ⇒ Bug	Reply to this comment
I just noticed that saving drafts when a session times out only works if use_vfs is enabled. But this is turned off by default and the configuration description only mentions attachments. I think we should have kept the old timeout code for the case that vfs is not available, but we should at least document that vfs is required for that feature now.