Uploaded image for project: 'Solder'
  1. Solder
  2. SOLDER-253

Integrate Catch with *EventBridge* classes

    Details

    • Type: Enhancement
    • Status: Resolved (View Workflow)
    • Priority: Critical
    • Resolution: Won't Fix
    • Affects Version/s: 3.0.0.Beta1
    • Fix Version/s: 3.1.0.Beta4
    • Component/s: Servlet
    • Labels:
      None
    • Estimated Difficulty:
      Low

      Description

      None of the EventBridge classes catch exceptions to pass to Seam Catch, thus the following code does not invoke any exception handlers:

         public void initial(@Observes @Path("NullPointer") @Initialized final HttpServletRequestContext ctx)
         {
            throw new NullPointerException();
         }
      

      I would expect the NPE to be passed to Catch for exception handling.

        Gliffy Diagrams

          Issue Links

            Activity

            Hide
            lightguard Jason Porter added a comment -

            Fixed with commit 02c851cdea1d129b94983c3817cbd798473db233

            Show
            lightguard Jason Porter added a comment - Fixed with commit 02c851cdea1d129b94983c3817cbd798473db233
            Hide
            lightguard Jason Porter added a comment -

            We reverted this fix, and users will have to do their own catch event firing for now.

            Show
            lightguard Jason Porter added a comment - We reverted this fix, and users will have to do their own catch event firing for now.
            Hide
            lightguard Jason Porter added a comment -

            I'm not sure if this is actually a bug, we had it fixed, but then had to revert it. I don't remember why right now though.

            Show
            lightguard Jason Porter added a comment - I'm not sure if this is actually a bug, we had it fixed, but then had to revert it. I don't remember why right now though.
            Hide
            lightguard Jason Porter added a comment -

            When this was fixed it created SOLDER-251, which is more of a bug than this.

            Show
            lightguard Jason Porter added a comment - When this was fixed it created SOLDER-251 , which is more of a bug than this.
            Hide
            lightguard Jason Porter added a comment -

            Because this causes SOLDER-251 this issue will not be fixed. Users should handle their own exceptions in these cases, or use an interceptor.

            Show
            lightguard Jason Porter added a comment - Because this causes SOLDER-251 this issue will not be fixed. Users should handle their own exceptions in these cases, or use an interceptor.

              People

              • Assignee:
                Unassigned
                Reporter:
                lightguard Jason Porter
              • Votes:
                0 Vote for this issue
                Watchers:
                2 Start watching this issue

                Dates

                • Created:
                  Updated:
                  Resolved:

                  Development