Conversation
- create autoconfiguration for each client or server module, without componentscan - move integration tests to a subpackage to avoid autowiring problems with the problem modules
- add missing @bean annotations - move error and exception handlers to common spring module
- add WebClientFilter - add an interface ProblemResponseErrorHandler
- reference ProblemResponseErrorHandler interface where possible
|
@jpraet , could you review if the changes in the PR look good to you high-level? I've tested it locally in a Spring (non-Boot) project using the |
Looks good to me high-level. Thank you |
…spring boot functionality and have very little code
|
The PR is now ready for review. I made following changes, on top of the previous ones:
|
|
| <dependencies> | ||
| <dependency> | ||
| <groupId>org.eclipse.transformer</groupId> | ||
| <artifactId>org.eclipse.transformer.cli</artifactId> |
There was a problem hiding this comment.
I wonder if we could/should use the same approach for the other -jakarta modules in the project.
The old approach is indeed not very user-friendly from IDE / debugging point of view.
Maybe we should split this off to another PR.



Lots of refactoring to enable support for Spring without Spring Boot dependencies.
@Configuration(application contexts) insteadWebClientFilterextractingPROBLEM_FILTERfromAbstractProblemWebClientCustomizerProblemResponseErrorHandler@EnableProblemModule(beanValidation=true, swaggerRequestValidation=false, ...)to enable easy activation in non SB-environmentOther:
RoutingExceptionHandler(the split was bcNoResourceFoundExceptionwas new in SBv3)@Valueinstead of@ConfigurationPropertiesto cut dependency to spring bootTODO:
EnableProblemModuleintellij shows issues that beans aren't found, while tests doe work