[MNG-8082] Exceptions of proxied SessionScoped components are not wor…

…king correctly (apache#1449) Signed-off-by: Jonas Rutishauser <[email protected]>
olamy · Apr 23, 2024 · aae74df · aae74df
1 parent 860310b
commit aae74df
Show file tree

Hide file tree

Showing 2 changed files with 14 additions and 1 deletion.
diff --git a/maven-core/src/main/java/org/apache/maven/session/scope/internal/SessionScope.java b/maven-core/src/main/java/org/apache/maven/session/scope/internal/SessionScope.java
@@ -107,7  107,11 @@ public <T> Provider<T> scope(final Key<T> key, final Provider<T> unscoped) {
     private <T> T createProxy(Key<T> key, Provider<T> unscoped) {
         InvocationHandler dispatcher = (proxy, method, args) -> {
             method.setAccessible(true);
-            return method.invoke(getScopeState().scope(key, unscoped).get(), args);
             try {
                 return method.invoke(getScopeState().scope(key, unscoped).get(), args);
             } catch (InvocationTargetException e) {
                 throw e.getCause();
             }
         };
         Class<T> superType = (Class<T>) key.getTypeLiteral().getRawType();
         Class<?>[] interfaces = getInterfaces(superType);

diff --git a/maven-core/src/test/java/org/apache/maven/session/scope/SessionScopeProxyTest.java b/maven-core/src/test/java/org/apache/maven/session/scope/SessionScopeProxyTest.java
@@ -74,6  74,7 @@ void testProxiedSessionScopedBean() throws ComponentLookupException {
         assertNotNull(bean.myBean.getSession());
         assertNotNull(bean.myBean.getAnotherBean());
         assertSame(bean.myBean.getAnotherBean().getClass(), AnotherBean.class);
         assertThrows(TestException.class, () -> bean.myBean.throwException());
     }
 
     @Named
@@ -102,6  103,8 @@ interface BeanItf {
         Session getSession();
 
         BeanItf2 getAnotherBean();
 
         void throwException() throws TestException;
     }
 
     interface BeanItf2 {}
@@ -127,5  130,11 @@ public Session getSession() {
         public BeanItf2 getAnotherBean() {
             return anotherBean;
         }
 
         public void throwException() throws TestException {
             throw new TestException();
         }
     }
 
     static class TestException extends Exception {}
 }