symmetryinvestments / imap-d

D library for IMAP (JMAP is a work-in-progress but the basics work)
14 stars 10 forks source link

Use of Mailbox and string is inconsistent and not thought-through #12

Open Laeeth opened 4 years ago

otrho commented 4 years ago

I think this is at least partly referring to the mailbox hierarchical separator which we don't seem to support properly.

The character used is defined by the server instance, and is usually . or / but needs to be used properly. Our mailbox handling and specification needs to support this more generally.

E.g., Right now we can perform mailbox operations using Mailbox objects which may have names containing . or / and they'll be passed through to the server as is and probably not work as expected.

Laeeth commented 4 years ago

Yes. I just ran out of time before implementing those and utf7 mailbox names properly

otrho commented 3 years ago

OK, also sending strings should be abstracted as there are quoted "strings" and there are string literals:

{33}
potentially
multiline
strings

We should have a sendString method which could like a slice of strings and send them as a string literal.

otrho commented 3 years ago

Also -- not necessarily related to this issue but is another basic feature we should support -- IDs can be UIDs, sequence IDs and also 'sequence sets', e.g., 2:4 meaning sequence IDs 2, 3 and 4.