Skip to content

chore: bump Groovy version#13225

Merged
yuriyz merged 3 commits intomainfrom
jans-auth-issue_13120
Feb 16, 2026
Merged

chore: bump Groovy version#13225
yuriyz merged 3 commits intomainfrom
jans-auth-issue_13120

Conversation

@jgomer2001
Copy link
Copy Markdown
Contributor

@jgomer2001 jgomer2001 commented Feb 16, 2026

Prepare


Description

Target issue

closes #13120

Implementation Details


Test and Document the changes

  • Static code analysis has been run locally and issues have been fixed
  • Relevant unit and integration tests have been added/updated
  • Relevant documentation has been updated if any (i.e. user guides, installation and configuration guides, technical design docs etc)

Please check the below before submitting your PR. The PR will not be merged if there are no commits that start with docs: to indicate documentation changes or if the below checklist is not selected.

  • I confirm that there is no impact on the docs due to the code changes in this PR.

Summary by CodeRabbit

  • Documentation

    • Updated certificate authentication documentation with clarifications on automatic user registration during enrollment.
    • Improved documentation examples and formatting for enhanced clarity.
  • Chores

    • Updated Groovy dependency to version 4.0.30.

Signed-off-by: jgomer2001 <bonustrack310@gmail.com>
Signed-off-by: jgomer2001 <bonustrack310@gmail.com>
@mo-auto
Copy link
Copy Markdown
Member

mo-auto commented Feb 16, 2026

Snyk checks have passed. No issues have been found so far.

Status Scanner Critical High Medium Low Total (0)
Open Source Security 0 0 0 0 0 issues

💻 Catch issues earlier using the plugins for VS Code, JetBrains IDEs, Visual Studio, and Eclipse.

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Feb 16, 2026

No actionable comments were generated in the recent review. 🎉


📝 Walkthrough

Walkthrough

This PR updates product documentation in the certificate authentication plugin guide with textual refinements regarding automatic user enrollment and attribute mappings, and upgrades the Groovy dependency from version 4.0.15 to 4.0.30 in the Agama engine.

Changes

Cohort / File(s) Summary
Documentation Updates
docs/casa/plugins/cert-authn.md
Refined enrollment descriptions to clarify automatic user registration behavior, adjusted heading levels from H2 to H3, and updated certificate attribute mapping examples from quoted to unquoted values.
Dependency Updates
jans-auth-server/agama/engine/pom.xml
Upgraded org.apache.groovy:groovy from version 4.0.15 to 4.0.30.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~5 minutes

Possibly related issues

Possibly related PRs

Suggested labels

kind-dependencies, comp-agama, comp-jans-auth-server

Suggested reviewers

  • yurem
  • yuriyzz
🚥 Pre-merge checks | ✅ 5 | ❌ 1

❌ Failed checks (1 inconclusive)

Check name Status Explanation Resolution
Out of Scope Changes check ❓ Inconclusive The PR includes documentation changes (cert-authn.md) in addition to the Groovy version bump; while these align with the commit message 'docs: minor update (#13185)', they extend beyond the primary objective of #13120. Clarify whether the documentation changes are part of the scope for #13120 or should be addressed in a separate PR to maintain focus on the Groovy update objective.
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title 'chore: bump Groovy version' directly and concisely describes the main change—updating the Groovy dependency version in the project.
Description check ✅ Passed The description follows the template with all required sections completed: Prepare checklist marked, Target issue (#13120) linked, and documentation impact confirmed.
Linked Issues check ✅ Passed The PR successfully addresses issue #13120 by updating Groovy from version 4.0.15 to 4.0.30, fulfilling the objective to update the library to a newer available version.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Merge Conflict Detection ✅ Passed ✅ No merge conflicts detected when merging into main

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch jans-auth-issue_13120

Tip

Issue Planner is now in beta. Read the docs and try it out! Share your feedback on Discord.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@mo-auto mo-auto added area-documentation Documentation needs to change as part of issue or PR comp-agama Touching folder /agama comp-docs Touching folder /docs comp-jans-auth-server Component affected by issue or PR kind-dependencies Pull requests that update a dependency file labels Feb 16, 2026
@sonarqubecloud
Copy link
Copy Markdown

Quality Gate Passed Quality Gate passed for 'Jans authentication server parent'

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
0.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube Cloud

@yuriyz yuriyz merged commit 22d5aaa into main Feb 16, 2026
1 of 3 checks passed
@yuriyz yuriyz deleted the jans-auth-issue_13120 branch February 16, 2026 12:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-documentation Documentation needs to change as part of issue or PR comp-agama Touching folder /agama comp-docs Touching folder /docs comp-jans-auth-server Component affected by issue or PR kind-dependencies Pull requests that update a dependency file

Projects

None yet

Development

Successfully merging this pull request may close these issues.

chore(agama): update Groovy lib

3 participants