James Server
  1. James Server
  2. JAMES-949

SMTP Async I/O - Core command handler not found in config

    Details

    • Type: Bug Bug
    • Status: Resolved
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: Trunk
    • Fix Version/s: 3.0-M1
    • Component/s: SMTPServer
    • Labels:
      None
    • Environment:
      Windows XP
      James trunk rev. 898686

      Description

      Hi all,

      I wanted to use the SMTP Async I/O, so I commented the normal smtp stuff and uncommented "org.apache.james.smtpserver.mina.AsyncSMTPServer" in spring-beans.xml

      Well, start-up fails.

      I pin-pointed the issue to AsyncSMTPServer.java to the method prepareHandlerChain() (line 197)

      I've attached a patch to resolve this issue, however I've no clue if this is the correct way

      ----------------------------------------------------------------------------

          1. Eclipse Workspace Patch 1.0
            #P james-server-smtpserver-function
            Index: src/main/java/org/apache/james/smtpserver/mina/AsyncSMTPServer.java
            ===================================================================
          • src/main/java/org/apache/james/smtpserver/mina/AsyncSMTPServer.java (revision 898686)
            +++ src/main/java/org/apache/james/smtpserver/mina/AsyncSMTPServer.java (working copy)
            @@ -194,7 +194,7 @@
            if (handlerchainConfig.getString("[@coreHandlersPackage]") == null)
            handlerchainConfig.addProperty("[@coreHandlersPackage]", CoreCmdHandlerLoader.class.getName());
      • handlerChain = getLoader().load(ProtocolHandlerChainImpl.class, getLogger(), handlerConfiguration);
        + handlerChain = getLoader().load(ProtocolHandlerChainImpl.class, getLogger(), handlerchainConfig);
        handlerChain.configure(handlerchainConfig);

      }

      ---------------------------------------------------------------------------

        Activity

        Mario Zsilak created issue -
        Hide
        Mario Zsilak added a comment -

        found the link to attach the patch in the proper way

        Show
        Mario Zsilak added a comment - found the link to attach the patch in the proper way
        Mario Zsilak made changes -
        Field Original Value New Value
        Attachment AsyncSMTPServer.patch [ 12430148 ]
        Hide
        Norman Maurer added a comment -

        Thx...

        could you please attach the patch as diff and grant rights via the checkbox

        Show
        Norman Maurer added a comment - Thx... could you please attach the patch as diff and grant rights via the checkbox
        Norman Maurer made changes -
        Assignee Norman Maurer [ norman ]
        Hide
        Mario Zsilak added a comment -

        Actually the patch (attached) is a SVN diff.
        Where is this checkbox which you mentioned? I did this "granting to ASF" .. Is this the correct one?

        Show
        Mario Zsilak added a comment - Actually the patch (attached) is a SVN diff. Where is this checkbox which you mentioned? I did this "granting to ASF" .. Is this the correct one?
        Hide
        Norman Maurer added a comment -

        Applied... thx again

        Show
        Norman Maurer added a comment - Applied... thx again
        Norman Maurer made changes -
        Status Open [ 1 ] Resolved [ 5 ]
        Fix Version/s 3.0-M1 [ 12314294 ]
        Resolution Fixed [ 1 ]
        Mark Thomas made changes -
        Workflow jira [ 12488119 ] Default workflow, editable Closed status [ 12566629 ]
        Mark Thomas made changes -
        Workflow Default workflow, editable Closed status [ 12566629 ] jira [ 12581811 ]

          People

          • Assignee:
            Norman Maurer
            Reporter:
            Mario Zsilak
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development