tkupka

Results 5 comments of tkupka

Hello Josh, the proposed solution is feasible but it looks slightly overcomplicated with custom forwarding URL which is responsible for rendering the UI. I looked through the code and the...

OK I see your point it makes no sense to add new dependency just for that. Looking into `opensaml-saml-impl` I see that they similar approach with velocity template(s). Would it...

``` org.openrewrite.java.Java17Parser$Builder.build(Java17Parser.java:96) org.openrewrite.java.Java17Parser$Builder.build(Java17Parser.java:63) org.openrewrite.java.internal.template.JavaTemplateParser.compileTemplate(JavaTemplateParser.java:257) org.openrewrite.java.internal.template.JavaTemplateParser.lambda$parseBlockStatements$9(JavaTemplateParser.java:176) org.openrewrite.java.internal.template.JavaTemplateParser.lambda$cacheIfContextFree$14(JavaTemplateParser.java:287) org.openrewrite.java.internal.template.JavaTemplateParser.cache(JavaTemplateParser.java:308) org.openrewrite.java.internal.template.JavaTemplateParser.cacheIfContextFree(JavaTemplateParser.java:287) org.openrewrite.java.internal.template.JavaTemplateParser.parseBlockStatements(JavaTemplateParser.java:171) org.openrewrite.java.internal.template.JavaTemplateJavaExtension$1.maybeReplaceStatement(JavaTemplateJavaExtension.java:469) org.openrewrite.java.internal.template.JavaTemplateJavaExtension$1.visitNewClass(JavaTemplateJavaExtension.java:448) org.openrewrite.java.internal.template.JavaTemplateJavaExtension$1.visitNewClass(JavaTemplateJavaExtension.java:56) org.openrewrite.java.tree.J$NewClass.acceptJava(J.java:4449) org.openrewrite.java.tree.J.accept(J.java:59) org.openrewrite.TreeVisitor.visit(TreeVisitor.java:283) org.openrewrite.TreeVisitor.visit(TreeVisitor.java:184) org.openrewrite.java.JavaTemplate.apply(JavaTemplate.java:101) ... at org.openrewrite.gradle.isolated.ResultsContainer$1.lambda$preVisit$0(ResultsContainer.java:115) at org.openrewrite.gradle.isolated.ResultsContainer$1.preVisit(ResultsContainer.java:112) at org.openrewrite.gradle.isolated.ResultsContainer$1.preVisit(ResultsContainer.java:108) at org.openrewrite.gradle.isolated.ResultsContainer_1_JavaVisitor.preVisit(ResultsContainer_1_JavaVisitor.zig:102) at org.openrewrite.TreeVisitor.visit(TreeVisitor.java:280) at org.openrewrite.TreeVisitor.visitAndCast(TreeVisitor.java:366)...

This is printed before: ``` Error during rewrite dry run org.openrewrite.internal.RecipeRunException: java.lang.IllegalStateException: Unable to construct Java17Parser. at org.openrewrite.TreeVisitor.visit(TreeVisitor.java:334) at org.openrewrite.TreeVisitor.visit(TreeVisitor.java:184) at org.openrewrite.java.JavaTemplate.apply(JavaTemplate.java:101) at org.openrewrite.java.spring.security6.UpdateRequestCache$1.visitMethodInvocation(UpdateRequestCache.java:109) at org.openrewrite.java.spring.security6.UpdateRequestCache$1.visitMethodInvocation(UpdateRequestCache.java:60) at org.openrewrite.java.tree.J$MethodInvocation.acceptJava(J.java:3932) at org.openrewrite.java.tree.J.accept(J.java:59)...

I have updated the `Test` class with 2 methods if I comment out failing one then rewrite passes.