Closed Bug 6904 Opened 25 years ago Closed 25 years ago

IMAP: expunge on close when copy-to-trash

Categories

(MailNews Core :: Networking, defect, P3)

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: jgmyers, Assigned: jefft)

References

Details

IS is having disk space problems attributable in part to unexpunged \Deleted
messages belonging to users who use the copy-to-trash delete model.

When using the copy-to-trash model, Communicator should always expunge folders
upon closing/shutdown.
QA Contact: 4080 → 4098
All folders, period; or just all folders with an open connection? The former
seems like it could take a lot of time for some types of users.
Preferably, all folders for which the client has previously seen deleted
messages.  For example, client could keep a "has deleted messages" bit in the
cache.  Just folders with an open connection would be better than nothing.
Assignee: mscott → jefft
I remember jefft saying he wanted a crack at this.....=)

re-assigning.
Might we want to use Close instead of expunge?
Status: NEW → ASSIGNED
Yes, CLOSE-LOGOUT & CLOSE-SELECT is much faster that EXPUNGE-LOGOUT &
EXPUNGE-SELECT since there is no untagged responses.
The current code base does try its best to issue "close" command prior doing
another select or logout if the user is in delete-is-move-to-trash mode.
However, I couldn't mark this bug fix until alecf fixed the shutdown problem for
the account manager and all the imap message incoming servers. Currently, imap
message incoming server holds the cached connections.
A side note: we might want to have a timer proc associate with a cached
connection to close the selected mail folder prior to the server times out the
connection. Will this overkill?
Target Milestone: M8
moving off m8 list. Also, didn't alec fix the shutdown problem? Perhaps this is
fixed?
I don't know. Need to verify whether the shutdown problem fixed or not.
Target Milestone: M8 → M9
moving to m9, even if it is already fixed.
This is fixed but creates a big problem for undo delete message to work. There
is no way to undo expunged messages. I am undoing my fix until I found a better
way to solve the undo problem.
This should happen at shut-down, at the same time as we expunged the inbox in
4.5., after which undo isn't possible. We certainly shouldn't expunge after
every copy to the trash.
Moving all Mail/News Networking bugs to Mail/News Networking-Mail

This may re-open previously Verified bugs due to a Bugzilla bug...if so, I will
fix those bugs.
Target Milestone: M9 → M10
M10 ... Don't know how to hook up expunge when shutting down the app yet.
Target Milestone: M10 → M12
moving to m12
Blocks: 11091
(target milestone is M11 or M12 - add to mail beta tracking bug)
Change QA Contact to me since it's my feature now.
QA Contact: scurtis → huang
Change QA Contact to me since it's my feature now.
Blocks: 18471
Status: ASSIGNED → RESOLVED
Closed: 25 years ago
Resolution: --- → FIXED
We only send close command to implicit expunge deleted message when shutdown.
This is needed because of the undo functionality. We need to allow users to undo
deleted message even when the connection was closed due to the time out or
maximum connections allowed per server. nsImapIncomingServer.cpp
nsImapProtocol.cpp modified.
Status: RESOLVED → VERIFIED
Verified on All platforms: Linux, WinNT & Mac 12-15-08/09-M12 commercial build:
From the application GUI interface, IMAP expunged "deleted message" after exit
(close/shutdown)the application.
From the IMAP log, still not find the "CLOSE" comment in the IMAP log file.
I am still marking as verified for this bug.
But, will add comments for bug#17716 for adding "CLOSE" comment for keep track
for this bug.
typo! I mean "CLOSE" command for IMAP log file.
No longer blocks: 18471
Product: MailNews → Core
Product: Core → MailNews Core
You need to log in before you can comment on or make changes to this bug.