Marco Collovati

Results 115 comments of Marco Collovati

The only reason I can see against `null` parent is that the provided URLs might not cover all possible location for parent classes, so a specific class cannot be loaded...

The Surefire plugin for in-process execution uses a custom class loader, temporarily set as the current thread context class loader. The classloader is built on project dependencies (` getProject().getArtifacts()`). ```java...

If I got it correctly, it uses the platform classloader as parent. https://github.com/apache/maven-surefire/blob/4281a9bdd41a5cc4f17f4ed101ee9e8a1a089aa5/surefire-booter/src/main/java/org/apache/maven/surefire/booter/Classpath.java#L120-L136 https://github.com/apache/maven-surefire/blob/master/surefire-booter/src/main/java/org/apache/maven/surefire/booter/IsolatedClassLoader.java

Looking at Quarkus `AwtProcessor` it seems that there have been some changes in GraalVM 23 related to AWT. For example this block of code https://github.com/quarkusio/quarkus/blob/4b8a27a5d591841d579a1b8ea1041b5fa481e8cf/extensions/awt/deployment/src/main/java/io/quarkus/awt/deployment/AwtProcessor.java#L276-L287 that points to https://github.com/oracle/graal/issues/4921. Link...

I mentioned the Quarkus class only because the hints in Flow were updated based on it

`Version` is specific for Flow. To get the Vaadin version you can use `Platform.getVaadinVersion()`

The `resetRolePrefix(Object source, Class type)` gets called potentially for two object types: `SecurityContextHolderAwareRequestWrapper` and `SecurityContextHolderAwareRequestFilter`. So we need to register reflection hints for these classes to make sure that fields...

Maybe not so strange. I suppose Spring never needs reflection on those classes

My opinion is that we need the hint on our side because we are accessing a field on the spring classes by using reflection. I suppose Spring itself does not...

![image](https://github.com/vaadin/hilla/assets/4648894/ec79dd5e-3d43-4572-a5a9-417556ee11f4)