Skip to content

[NAE-1617] Authority refactor - #477

Draft
renczesstefan wants to merge 9 commits into
release/7.0.0from
NAE-1617_v7
Draft

renczesstefan wants to merge 9 commits into
release/7.0.0from
NAE-1617_v7

Conversation

@renczesstefan

@renczesstefan renczesstefan commented Sep 21, 2026

Copy link
Copy Markdown
Member

Description

Implements NAE-1617

Dependencies

No new dependencies were introduced.

Third party dependencies

No new dependencies were introduced.

Blocking Pull requests

There are no dependencies on other PR.

How Has Been This Tested?

This was tested manually and with unit tests.

Test Configuration

Name Tested on
OS macOS Tahoe 26.5.2
Runtime Java 21
Dependency Manager Maven 3.9.9n
Framework version Spring Boot 3.4.4
Run parameters
Other configuration

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • My changes have been checked, personally or remotely, with @machacjozef
  • I have commented my code, particularly in hard-to-understand areas
  • I have resolved all conflicts with the target branch of the PR
  • I have updated and synced my code with the target branch
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing tests pass locally with my changes:
    • Lint test
    • Unit tests
    • Integration tests
  • I have checked my contribution with code analysis tools:
  • I have made corresponding changes to the documentation:
    • Developer documentation
    • User Guides
    • Migration Guides

…izations` annotations, implement `BaseAuthorizationServiceAspect`, extend Authority management, and update dependencies.
…annotation support, and comprehensive documentation.
…rove `@Authorize` annotation functionality, add `AuthorityDto`, refactor REST endpoints, and update documentation.
…er handling, enhance `AuthorityConfigurationProperties` with documentation, improve `AuthorityRunner` authority creation logic, and extend `AuthorizingObject` enum.
…base to standardize authorization handling and improve maintainability. Extend `AuthorizingObject` enum and refactor related methods, services, and documentation.
@coderabbitai

coderabbitai Bot commented Sep 21, 2026

Copy link
Copy Markdown

Important

Draft PR not reviewed

Draft PRs are not automatically reviewed by default.

  • Trigger a manual review

To automatically review draft PRs, update your CodeRabbit configuration:

reviews:
  auto_review:
    drafts: true

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.

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
3.7% Coverage on New Code (required ≥ 60%)

See analysis details on SonarQube Cloud


Optional<Authority> findOptionalByName(String id);

@Cacheable("defaultUserAuthoritiesCache")
@Cacheable("defaultUserAuthoritiesCache")
Set<Authority> getDefaultUserAuthorities();

@Cacheable("defaultAnonymousAuthoritiesCache")
@Cacheable("defaultAnonymousAuthoritiesCache")
Set<Authority> getDefaultAnonymousAuthorities();

@Cacheable("defaultAdminAuthoritiesCache")

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants