Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Grails 7 - Spring Security - Cleanup #1046

Merged
merged 27 commits into from
Dec 17, 2024
Merged

Grails 7 - Spring Security - Cleanup #1046

merged 27 commits into from
Dec 17, 2024

Conversation

matrei
Copy link
Contributor

@matrei matrei commented Dec 17, 2024

No description provided.

codeconsole and others added 27 commits November 22, 2024 15:48
@codeconsole
Use grails-bom
14f6164
@codeconsole
cleanup
704bf81
@codeconsole
Update SecurityFilterPosition.java to match Spring Security 6.1
ce3e10b
@codeconsole
Merge branch '7.0.x' into 7.0.x-grails-bom
7b39b36
@jdaugherty
Always run the tests
d586c81
@jdaugherty
Merge remote-tracking branch 'codeconsole/7.0.x-grails-bom' into 7.0.x
5bda9c4
@jdaugherty
Use compileJava.options.release to set the java version
bf3f0e3
@jdaugherty
Use asset plugin version 5.0.4
2fde71f
@jdaugherty
Switch to ContainerGebSpec
d4bb98b
@jdaugherty
Remove GebConfig.groovy files
a1da663
@jdaugherty
Remove ReportingGebSpec
891308e
@jdaugherty
add --continue so test runs don't immediately fail.
dc24f94
@jdaugherty
Do not fail-fast if one of the gradle checks fails
ea3efe3
@jdaugherty
Fix layout name
b6de3cb
@jdaugherty
Adding missing events plugin
1d1ec55
@jdaugherty
Test reset cleanup
212dd3a
@jdaugherty
Change connection init to download()
4ae969c
@jdaugherty
remove geb.env
720d9c3
@jdaugherty
Update url for ContainerGebSpec
5c69edc
@jdaugherty
Apply sitemesh filter after the FormContentFilter
624e84b
@jdaugherty
Ensure data is built and not rolled back
360eaf2
@jdaugherty
Remove going to index
9c90a44
@matrei
fix(deps): remove all micronaut-inject-groovy dependencies
62c063a 
Micronaut is removed from Grails 7.
@matrei
fix(deps): add gsp dependency to test app
2b6313d
@matrei
fix: add default 'main' layout to plugin
8129ef1 
The default `main` layout is referenced in `login/auth.gsp` and `login/denied.gsp` but it is not included in the plugin.

This commit adds the `main` layout to the plugin so it can be found even if there is not such layout in the application.
@matrei
test: use setup() fixture method in test
d3ce674
@matrei
test: add views in test app
04c5c87 
These views are referenced in the `UrlMappings`.
jdaugherty
jdaugherty approved these changes Dec 17, 2024
View reviewed changes
Copy link
Contributor

@jdaugherty jdaugherty left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I suggest we merge this to get the plugin working and revisit the notFound, documentation, & q2start app updates

@matrei matrei changed the title Matrei/jdaugherty 7.0.x Grails 7 - Spring Security - Cleanup Dec 17, 2024
@matrei matrei merged commit 81a805f into 7.0.x Dec 17, 2024
14 checks passed
@matrei matrei deleted the matrei/jdaugherty-7.0.x branch December 17, 2024 12:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jdaugherty jdaugherty jdaugherty approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@matrei @jdaugherty @codeconsole