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

          There are no Sub-Tasks for this issue.

              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:

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