This is an automated email from the ASF dual-hosted git repository.
rombert pushed a commit to branch master
in repository
https://gitbox.apache.org/repos/asf/sling-org-apache-sling-servlets-resolver.git
The following commit(s) were added to refs/heads/master by this push:
new 01da40a SLING-13016 - RequestWrapper causes NPEs when serving bundled
scripts if 'forceResourceType' is not set (#61)
01da40a is described below
commit 01da40a47c698659e5f6cbdf20b69f359f8a1b2f
Author: Robert Munteanu <[email protected]>
AuthorDate: Fri Nov 28 16:12:45 2025 +0100
SLING-13016 - RequestWrapper causes NPEs when serving bundled scripts if
'forceResourceType' is not set (#61)
Add back the null check.
---
.../sling/servlets/resolver/internal/bundle/RequestWrapper.java | 4 +++-
1 file changed, 3 insertions(+), 1 deletion(-)
diff --git
a/src/main/java/org/apache/sling/servlets/resolver/internal/bundle/RequestWrapper.java
b/src/main/java/org/apache/sling/servlets/resolver/internal/bundle/RequestWrapper.java
index d46a6c1..049bbda 100644
---
a/src/main/java/org/apache/sling/servlets/resolver/internal/bundle/RequestWrapper.java
+++
b/src/main/java/org/apache/sling/servlets/resolver/internal/bundle/RequestWrapper.java
@@ -41,7 +41,9 @@ public class RequestWrapper extends
SlingJakartaHttpServletRequestWrapper {
if (resource == null) {
return null;
}
- if (options != null && options.getForceResourceType().isEmpty()) {
+ if (options != null
+ && (options.getForceResourceType() == null
+ || options.getForceResourceType().isEmpty())) {
options.setForceResourceType(resource.getResourceType());
}
RequestDispatcherOptions processedOptions = processOptions(options);