Qpid
  1. Qpid
  2. QPID-2608

AMQPException messages contain duplicate AMQConstant text

    Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Minor Minor
    • Resolution: Fixed
    • Affects Version/s: 0.5, 0.6
    • Fix Version/s: 0.7
    • Component/s: Java Broker, Java Client
    • Labels:
      None

      Description

      The AMQPException class builds the message string in the constructor. Nested eceptions can cause the mssage text taken from AMQPConstant to be duplicated (possibly several times). The message should be build in the toString method instead.

        Activity

        Hide
        Andrew Kennedy added a comment -

        Move message extraction from AMQPConstant to toString and added an AMQPSecurityException subclass

        Show
        Andrew Kennedy added a comment - Move message extraction from AMQPConstant to toString and added an AMQPSecurityException subclass
        Hide
        Andrew Kennedy added a comment -

        Removed reference to AMQSecurityException

        Show
        Andrew Kennedy added a comment - Removed reference to AMQSecurityException
        Hide
        Robbie Gemmell added a comment -

        The override on toString() to clean up the error code cloning in cloneForCurrentThread() doesnt print the class name as the standard Throwable output would, it would be good to put this back in.

        Show
        Robbie Gemmell added a comment - The override on toString() to clean up the error code cloning in cloneForCurrentThread() doesnt print the class name as the standard Throwable output would, it would be good to put this back in.
        Hide
        Andrew Kennedy added a comment -

        Updated patch based on review comments

        Show
        Andrew Kennedy added a comment - Updated patch based on review comments
        Hide
        Andrew Kennedy added a comment -

        Updated ExternalACLJMXTest to reflect changes in the AMQException toString method when examining exception stacktraces

        Show
        Andrew Kennedy added a comment - Updated ExternalACLJMXTest to reflect changes in the AMQException toString method when examining exception stacktraces
        Hide
        Robbie Gemmell added a comment -

        Appled patch: 0001-QPID-2608-AMQPException-messages-contain-duplicate.patch

        Show
        Robbie Gemmell added a comment - Appled patch: 0001- QPID-2608 -AMQPException-messages-contain-duplicate.patch

          People

          • Assignee:
            Robbie Gemmell
            Reporter:
            Andrew Kennedy
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development