imap_close

[ghost]

Shouldn't imap_close be called after each operation?

[tedivm]

No, constantly closing and opening the connection would be a serious drain on resources for both the server and the client (in this case, whatever app is using Fetch). If you look at email clients that live on the desktop they all maintain their connection over the long term.

[ghost]

Doesn't this library naturally create a new connection across requests though? How does the connection stay persistent against a page reload for example?

[tedivm]

It doesn't. Each page request or run of the application has it's own connection.

A change to this application that may make sense is adding imap_close to the Server object's destructor function, but it certainly should not be done after each operation.

[ghost]

+1 for that

[ghost]

#62 should take care of this :)

[ghost]

#69 is the latest version, now that I'm using branches...