-
Notifications
You must be signed in to change notification settings - Fork 61
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Version 3.0.0 totally breaks even trivial applications. #169
Comments
I don't know whats your problem, as you didn't state it. Here, your example runs...?
|
Ah, I see. In your example, audit-logging is not included at all. |
Found it. We have springsec plugin dependency as provided in the plugin classpath to be able to compile the SpringSecurityRequestResolver. It seems this is enough that the ControllerMixin gets compiled into the Controller class. |
https://github.com/aadrian/grenderingz/blob/audit_logging/build.gradle#L64 As I described, the branch |
I’ll take a look. We are using the plugin in a very large 3.3.6 application with dozens of audited domains and dependencies. |
The https://github.com/robertoschwald/grails-audit-logging-plugin/pull/170 should fix this issue. I used the sample app in this ticket to verify the changes. |
Please use Version 3.0.1 |
@robertoschwald @longwa yes, with 3.0.1 it works again :) . |
grails-audit-logging-plugin
version 3.0.0 seems to totally break even trivial Grails 3.3.5 applications :( .Please see in this small example: https://github.com/aadrian/grenderingz/tree/audit_logging
After adding the plug-in, nothing works anymore aadrian/grenderingz@44eb326 :( .
Following the documentation and how this version of the plug-in it's supposed to be different from version 2.x, those are the only changes made that are visible in that commit diff.
Am I missing something? Or is the plug-in indeed broken with Grails 3.3.5?
Thank you.
The text was updated successfully, but these errors were encountered: