glassfish
  1. glassfish
  2. GLASSFISH-16251

Include the deployment exception when using REST to deploy

    Details

    • Type: New Feature New Feature
    • Status: Resolved
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 3.1
    • Fix Version/s: 3.1.1_dev
    • Component/s: rest-interface
    • Labels:
      None
    • Environment:

      All

      Description

      When you deploy via the rest interface the exception causing the deployment to fail is not in the payload. This makes it very difficult to report problems if they occur.

      To recreate:

      clone Seam Solder at https://github.com/seam/solder
      cd impl
      mvn integration-test -Dincontainer-glassfish-rest -Dtest=ServiceHandlerTest

      Or use the attached war via curl:

      curl -s -S -H 'Accept: application/xml' -X POST -F id=@org.jboss.seam.solder.test.serviceHandler.ServiceHandlerTest_test.war -F force=true http://localhost:4848/management/domain/applications/application

        Activity

        Hide
        Jason Lee added a comment -

        Fix committed (r45695). We were missing a 'return' keyword, which caused execution to go passed where it should, resulting in an incorrect error message and code being returned to the user.

        Show
        Jason Lee added a comment - Fix committed (r45695). We were missing a 'return' keyword, which caused execution to go passed where it should, resulting in an incorrect error message and code being returned to the user.

          People

          • Assignee:
            Jason Lee
            Reporter:
            lightguard
          • Votes:
            0 Vote for this issue
            Watchers:
            0 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved: