Plugin breaks Grails 3.2.11 about grails-melody-plugin HOT 14 CLOSED

Sorry, I do not reproduce the issue.
By using the sample project at https://github.com/evernat/grails-ssl it works correctly with Grails 3.2.11 and Java 1.8. https://localhost:8443/ and https://localhost:8443/monitoring are OK.
What would cause this sample project to break?

from grails-melody-plugin.

@evernat just add <g:link controller="test">foo</g:link> to index.gsp and it will start throwing the exception

from grails-melody-plugin.

This was also raised by me in grails repo: grails/grails-core#10715

from grails-melody-plugin.

I got the same bug.
Can be reproduced by line in BootStrap.init:
"some text".encodeAsLink()

As I remember, I have had got same error in Grails 3.2.10

@evernat Any workarounds?

from grails-melody-plugin.

@graemerocher suggested in grails/grails-core#10715:

My guess is something to do with the Groovy version, Groovy 2.4.11 is used in 3.2.11

from grails-melody-plugin.

As said in grails/grails-core#10715 (comment), it seems to be caused by a regression between groovy 2.4.8 and groovy 2.4.9.
To workaround the issue, you may want to add compile 'org.codehaus.groovy:groovy-all:2.4.8' in the dependencies of the build.gradle file of your app.

from grails-melody-plugin.

@evernat using groovy 2.4.8 with grails 3.2 acceptable only as a workaround.
Old version of groovy as long term solution -- bad idea. For example, a performance of Groovy 2.4.10 has been improved by 15-20 %.

Do you plan to fix the plugin?

from grails-melody-plugin.

I recommend reporting the issue to the Groovy issue tracker grails/grails-core#10715

from grails-melody-plugin.

@evernat would it be possible to test the following:

• upgrade the current plugin to grails 3.2.11 (it's 3.1.6 right now https://github.com/javamelody/grails-melody-plugin/blob/master/gradle.properties#L1)
• build a snapshot
• test that snapshot version with a currently failing example project

Perhaps the issue goes away if the plugin itself is build by groovy > 2.4.9 ?

from grails-melody-plugin.

@jglapa @sergiomichels
It seems that building the grails-melody-plugin with the following in gradle.properties of the plugin fixes this issue:

grailsVersion=3.2.11
grailsWrapperVersion=1.0.0
gradleWrapperVersion=3.4.1

But I am not sure of what I was doing in the plugin for testing this. Sergio, could you check that this change in the plugin will fix the error displayed by the sample project: https://github.com/evernat/grails-ssl ?

from grails-melody-plugin.

@evernat grails/grails-core#10715 (comment)

from grails-melody-plugin.

Created #50

from grails-melody-plugin.

The Pull Request (50) above does indeed fix the problem described in this ticket under Grails 3.3.0, but a new problem emerges: now melody does not load at startup. The only other change is upgrading the java-melody-core dependency to 1.68.1.

Here are the messages at application startup, indicating the problem.

2017-08-02 02:42:16.422  INFO 11581 --- [  restartedMain] ationConfigEmbeddedWebApplicationContext : Refreshing org.springframework.boot.context.embedded.AnnotationConfigEmbeddedWebApplicationContext@4443b126: startup date [Wed Aug 02 02:42:16 UTC 2017]; root of context hierarchy
2017-08-02 02:42:18.038  WARN 11581 --- [  restartedMain] org.grails.plugins.CorePluginFinder      : [GrailsPluginManager] Core plugin [grails.melody.plugin.GrailsMelodyPluginGrailsPlugin] not found, resuming load without..

from grails-melody-plugin.

PR #50 is merged. Thanks to all and in particular to @graemerocher and @bassmartin.

from grails-melody-plugin.