james-server-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Robert Burrell Donkin (JIRA)" <server-...@james.apache.org>
Subject [jira] Commented: (IMAP-71) Design: Consolidate Sessions
Date Sat, 09 May 2009 13:30:46 GMT

    [ 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: server-dev-unsubscribe@james.apache.org
For additional commands, e-mail: server-dev-help@james.apache.org

View raw message