This is an automated email from the ASF dual-hosted git repository. cziegeler pushed a commit to branch master in repository https://gitbox.apache.org/repos/asf/sling-org-apache-sling-resourceresolver.git
commit ab29b3019110a2be304d8204c0adbb55e0d28a61 Author: Carsten Ziegeler <[email protected]> AuthorDate: Mon Nov 13 16:40:29 2023 +0100 SLING-12149 : ResourceResolver: Illegal mode passthrough for resource provider null --- .../resourceresolver/impl/providers/ResourceProviderInfo.java | 7 +++++-- 1 file changed, 5 insertions(+), 2 deletions(-) diff --git a/src/main/java/org/apache/sling/resourceresolver/impl/providers/ResourceProviderInfo.java b/src/main/java/org/apache/sling/resourceresolver/impl/providers/ResourceProviderInfo.java index 6f475dd..e075b5e 100644 --- a/src/main/java/org/apache/sling/resourceresolver/impl/providers/ResourceProviderInfo.java +++ b/src/main/java/org/apache/sling/resourceresolver/impl/providers/ResourceProviderInfo.java @@ -71,7 +71,7 @@ public class ResourceProviderInfo implements Comparable<ResourceProviderInfo> { try { aType = AuthType.valueOf(authType); } catch ( final IllegalArgumentException iae) { - logger.error("Illegal auth type {} for resource provider {}", authType, name); + logger.error("Illegal auth type {} for resource provider {} ({})", authType, name, ref); } this.authType = aType; this.modifiable = c.convert(ref.getProperty(ResourceProvider.PROPERTY_MODIFIABLE)).to(boolean.class); @@ -84,9 +84,12 @@ public class ResourceProviderInfo implements Comparable<ResourceProviderInfo> { try { mode = Mode.valueOf(modeValue); } catch ( final IllegalArgumentException iae) { - logger.error("Illegal mode {} for resource provider {}", modeValue, name); + logger.error("Illegal mode {} for resource provider {} ({})", modeValue, name, ref); } this.mode = mode; + if ( !path.startsWith("/") ) { + logger.error("Path {} does not start with / for resource provider {} ({})", path, name, ref); + } } public boolean isValid() {
