-
Notifications
You must be signed in to change notification settings - Fork 162
feat(config-api): feature wise admin scope for endpoints#11633 #12736
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
Open
pujavs
wants to merge
67
commits into
main
Choose a base branch
from
jans-config-fix
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from all commits
Commits
Show all changes
67 commits
Select commit
Hold shift + click to select a range
e395466
feat(config-api): user mgt serach enhancehment
pujavs 85e53bb
feat(config-api): user mgt search filter enhancement
pujavs f3b779f
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs ed8dade
feat(config-api: user mgt enhancement
pujavs 3d18e38
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs 8220001
feat(config-api): agma endpoint metadata changes
pujavs 1d3de61
Merge branch 'main' into jans-config-fix
yuriyz e81a37b
feat(config-api): agama deployment metadata type fix for deployment #…
pujavs ac00d58
Merge branch 'jans-config-fix' of https://github.com/JanssenProject/j…
pujavs 093c1ad
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs c974095
feat(config-api): agama deployment metadata type fix for deployment #…
pujavs 435f3f7
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs d5c59c6
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs e74516a
docs(config-api): swagger spec update
pujavs e5646c0
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs 3eeac46
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs 338fc64
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs ab8c33e
feat(config-api): Super scope access to access endpoint#11633
pujavs 89a0b52
feat(config-api): feature wise admin scope for endpoints#11633
pujavs efcbd50
feat(config-api): sync with main
pujavs 4cec68a
feat(config-api): feature wise admin scope for endpoints#11633
pujavs 04dc188
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs 66c4c31
feat(config-api): feat(config-api): feature wise admin scope for endp…
pujavs 5affabe
feat(config-api): feat(config-api): feature wise admin scope for endp…
pujavs 18ddc12
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs 826bb03
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs ae5bd05
feat(config-api): super admin scope for each functionality
pujavs 46ea700
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs 19a62fb
feat(config-api): syn with main
pujavs d316731
feat(config-api): feature admin scope - wip
pujavs 4672662
feat(config-api): sync with main
pujavs cfc358f
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs ebcf6f9
feat(config-api): admin scope wip
pujavs 4681d1b
feat(config-api): admin scope wip
pujavs 710a108
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs b06512a
feat(config-api): admin scopes for function - wip
pujavs cbce088
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs 24f4e25
feat(config-api): super admin scope - wip
pujavs fa46ba4
feat(config-api) scope wip
pujavs d07cf6c
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs d927d61
feat(config-api): sync with main
pujavs 286e58e
feat(config-api): super admin scope - wip
pujavs 6e00136
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs d79bf45
feat(config-api): super admin wip
pujavs 325d3df
feat(config-api): super admin scope - wip
pujavs b198cda
feat(config-api): user serach by mobile enhancement
pujavs 82b8c8a
Merge branch 'main' into jans-config-fix
yuriyz 122db23
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs 922bd9a
feat(config-api): super admin scope
pujavs d2c97c8
Merge branch 'jans-config-fix' of https://github.com/JanssenProject/j…
pujavs aec2583
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs 38319f3
feat(config-api): feature super admin wip
pujavs b7b1e98
feat(config-api): feature wise admin scope for endpoints#11633
pujavs 5224efa
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs b203b30
feat(config-api): config-api admin scope
pujavs d947f9e
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs a4257d8
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs d4f4eb7
feta(config-api): super feature scope
pujavs 366df95
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs c23a27c
t status
pujavs f704eac
feat(config-api): admin scope changes as per coderrabit
pujavs 950608e
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs b4da651
feat(config-api): feature wise admin scope
pujavs c1b7599
Merge branch 'main' of https://github.com/JanssenProject/jans into ja…
pujavs 5534fcd
feat(config-api): feature admin scope
pujavs 5c447c2
feat(config-api): feature admin scope
pujavs 8411c17
feat(config-api): feature admin scope
pujavs File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Some comments aren't visible on the classic Files Changed page.
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🧩 Analysis chain
🏁 Script executed:
Repository: JanssenProject/jans
Length of output: 3798
🏁 Script executed:
Repository: JanssenProject/jans
Length of output: 1845
🏁 Script executed:
Repository: JanssenProject/jans
Length of output: 6097
OPENID_READ_ACCESS uses inconsistent scope naming pattern within the same constants file.
The
OPENID_READ_ACCESSconstant (line 106) uses dash notation (openid-read), while resource-specific scopes in the same file likeOPENID_CLIENTS_READ_ACCESS(line 58) use dot notation (clients.readonly). Although the comment "Feature Scope" suggests this distinction is intentional, the inconsistency withinApiAccessConstants.javais problematic. Either document this design decision clearly with inline comments explaining why feature-level scopes use dashes while resource scopes use dots, or standardize on a single naming convention across all OPENID scopes.🤖 Prompt for AI Agents