Uploaded image for project: 'Magnolia REST Framework'
  1. Magnolia REST Framework
  2. MGNLREST-681

AssetReferenceResolver#resolve catches different exception than it's actually thrown in JcrAssetProvider#getNodeByIdentifier

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Neutral Neutral
    • 2.2.19
    • 2.2.15
    • delivery
    • None
    • DevX 38, DevX 39
    • 3
    • Yes

      info.magnolia.rest.reference.dam.AssetReferenceResolver has this code:

      } catch (IllegalArgumentException e) {
          log.error("Key {} is not valid Asset key", item);
      }
      

      but info.magnolia.dam.jcr.JcrAssetProvider from DAM Module has this code:

      } catch (RepositoryException e) {
          throw new IllegalItemKeyException(itemKey, this, e.getMessage());
      }
      

      so exceptions thrown for invalid identifier are actually never caught. Which results of HTTP 500 response.

        Acceptance criteria

              vrg Valeria Robles-Garzon
              msiradzki Marzena Sieradzka
              DeveloperX
              Votes:
              0 Vote for this issue
              Watchers:
              1 Start watching this issue

                Created:
                Updated:
                Resolved:
                Work Started:

                  Bug DoR
                  Task DoD

                    Estimated:
                    Original Estimate - Not Specified
                    Not Specified
                    Remaining:
                    Remaining Estimate - Not Specified
                    Not Specified
                    Logged:
                    Time Spent - 0.75d
                    0.75d