Uploaded image for project: 'PUBLIC - Liferay Portal Community Edition'
  1. PUBLIC - Liferay Portal Community Edition
  2. LPS-12613

isPermanent flag on UnavailableException is not respected

    Details

    • Branch Version/s:
      6.0.x, 5.2.x
    • Backported to Branch:
      Committed

      Description

      In render_portlet.jsp you will find the following code :

      if (portlet.isActive() && access && supportsMimeType) {
      try {
      invokerPortlet.render(renderRequestImpl, renderResponseImpl);
      ....
      } catch (UnavailableException ue) {
      portletException = true;
      PortletInstanceFactoryUtil.destroy(portlet);
      }

      It will actually destroy the portlet when an UnavailableException is thrown while per spec, it should only do this when the UnavailableException is permanent

      Here is the relevant excerpt from jsr286

      PLT.5.4.7 Exceptions During Request Handling

      If a permanent unavailability is indicated by the UnavailableException, the portlet container must remove the portlet from service immediately, call the portlet's destroy method, and release the portlet object.xxiv A portlet that throws a permanent
      15 UnavailableException must be considered unavailable until the portlet application containing the portlet is restarted.
      When temporary unavailability is indicated by the UnavailableException, then the portlet container may choose not to route any requests to the portlet during the time period of the temporary unavailability

        Attachments

          Issue Links

            Activity

              People

              Assignee:
              support-lep@liferay.com SE Support
              Reporter:
              jelmer Jelmer Kuperus (Inactive)
              Participants of an Issue:
              Recent user:
              Esther Sanz
              Votes:
              0 Vote for this issue
              Watchers:
              0 Start watching this issue

                Dates

                Created:
                Updated:
                Resolved:
                Days since last comment:
                10 years, 7 weeks, 2 days ago

                  Packages

                  Version Package
                  --Sprint - SP
                  6.1.0 CE RC1