[ 
https://issues.apache.org/jira/browse/IMAP-71?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=12707665#action_12707665
 ] 

Robert Burrell Donkin commented on IMAP-71:
-------------------------------------------

The API should be revised so that MailboxSession is required for every method. 
This will enable the backend to obtain the user context.

> Design: Consolidate Sessions
> ----------------------------
>
>                 Key: IMAP-71
>                 URL: https://issues.apache.org/jira/browse/IMAP-71
>             Project: JAMES Imap
>          Issue Type: Improvement
>          Components: Mailbox, Protocol
>    Affects Versions: 0.1
>            Reporter: Robert Burrell Donkin
>             Fix For: 0.1
>
>
> ATM two sessions are used - an ImapSession for the IMAP protocol layers, and 
> a MailboxSession for the mailbox layers. This is probably a mistake. The API 
> would be easier to extend if protocol and mailbox used the same session. 
> Unfortunately, ImapSession is coupled to a number of quite IMAP specific 
> classes. Improvements to the design in this area (eg. simplification of the 
> inefficient notification system) would probably allows this to be refactored 
> but for the moment, seems better to push methods up into MailboxSession, and 
> MailboxSession up into the API. ImapSession could then extend MailboxSession.

-- 
This message is automatically generated by JIRA.
-
You can reply to this email to add a comment to the issue online.


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to