Details

    • Type: Bug Bug
    • Status: Closed
    • Priority: Major Major
    • Resolution: Fixed
    • Affects Version/s: 2.1.6, 2.1.8
    • Fix Version/s: 2.1.8
    • Component/s: Other
    • Labels:
      None
    • Environment:

      Freemarker

      Description

      We render sub-portions of our pages using s.action. Unfortunately, when one of these fails, rather than ending up with a 500 error (and in our own error/exception handling code), the user gets a partially rendered page. This is because ActionComponent.executeAction() catches and logs all exceptions, but never rethrows them:

      } catch (Exception e) {
      String message = "Could not execute action: " + namespace + "/" + actualName;
      LOG.error(message, e);
      }

      There could be a parameter added of course to request that exceptions be swallowed for case where a partial render might be fine, or executeResult is false and they don't care if it fails, but it does seem like that would be better handled explicitly in the action being invoked instead.

        Activity

        Jasper Rosenberg created issue -
        Wes Wannemacher made changes -
        Field Original Value New Value
        Resolution Fixed [ 1 ]
        Fix Version/s 2.0.15 [ 21903 ]
        Status Open [ 1 ] Resolved [ 5 ]
        Assignee Wes Wannemacher [ wesw@wantii.com ]
        Wes Wannemacher made changes -
        Affects Version/s 2.1.8 [ 21920 ]
        Affects Version/s 2.1.7 [ 21907 ]
        Jeff Turner made changes -
        Project Import Mon Feb 01 01:17:42 UTC 2010 [ 1264987062082 ]
        Lukasz Lenart made changes -
        Status Resolved [ 5 ] Closed [ 6 ]

          People

          • Assignee:
            Wes Wannemacher
            Reporter:
            Jasper Rosenberg
          • Votes:
            0 Vote for this issue
            Watchers:
            1 Start watching this issue

            Dates

            • Created:
              Updated:
              Resolved:

              Development