-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
RBAC Phase 1 #19723
Merged
Merged
RBAC Phase 1 #19723
Changes from all commits
Commits
Show all changes
85 commits
Select commit
Hold shift + click to select a range
01b5cf0
partial implementation for OLS Phase 1
legrego 7cee640
Allow Saved Objects Client to be wrapped
legrego 0afd1c1
Add placeholder "kibana.namespace" configuration property
legrego 30e86d1
revert changes to saved objects client
legrego 936180b
Remove circular dependency
legrego 4d4f946
Removing namespace setting, we're using xpack.security.rbac.application
kobelb df569df
Adding config.getDefault
kobelb 9979fb9
Expose SavedObjectsClientProvider on the server for easy plugin consu…
legrego 646a80a
migrate x-pack changes into kibana
legrego d679cf5
Beginning to use the ES APIs to insert/check privileges (#18645)
kobelb 36e7a67
Adding built-in types and alphabetizing (#19306)
kobelb 06eb784
Filtering out non-default resource Kibana privileges (#19321)
kobelb 767fb27
Removing unused file
kobelb 203ec3e
Adding kibana_rbac_dashboard_only_user to dashboard only mode roles (…
kobelb d818cc6
Adding create default roles test (#19505)
kobelb 813a816
Merge remote-tracking branch 'upstream/master' into rbac-phase-1
kobelb 66226d6
Merge remote-tracking branch 'upstream/master' into rbac-phase-1
kobelb d8d9810
RBAC - SecurityAuditLogger (#19571)
kobelb 3e8e694
RBAC Integration Tests (#19647)
kobelb 74b8486
Merge remote-tracking branch 'upstream/master' into rbac-phase-1
kobelb 1231c70
Fixing "conflicts" after merging master
kobelb 99d70b9
Removing some white-space differences
kobelb db18d1e
Deleting files that got left behind in a merge
kobelb d793056
Adding the RBAC API Integration Tests
kobelb 7f2c9b0
SavedObjectClient.find filtering (#19708)
kobelb b6093bc
Trying to isolate cause of rbac test failures
kobelb 4abf5ed
Adding .toLowerCase() to work around capitalization issue
kobelb d951a20
No longer exposing the auditLogger, we don't need it like that right now
kobelb 8920425
Removing some unused code
kobelb 91d04e4
Removing defaultSettings from test that doesn't utilize them
kobelb 430d72c
Fixing misspelling
kobelb 7977f00
Don't need an explicit login privilege when we have them all
kobelb 60d5917
Removing unused code, fixing misspelling, adding comment
kobelb 4b3c6ba
Putting a file back
kobelb 117b0d4
No longer creating the roles on start-up (#19799)
kobelb c42635b
Removing kibana_rbac_dashboard_only_user from dashboard only role
kobelb e887076
Fixing small issue with editing Kibana privileges
kobelb a76e4a5
[RBAC Phase 1] - Update application privileges when XPack license cha…
legrego 7ef5850
RBAC Legacy Fallback (#19818)
kobelb 9941eb2
Fixing checkLicenses tests since we added RBAC
kobelb 8667ebd
[Flaky Test] - wait for page load to complete (#19895)
legrego b0e6998
[Flaky Test] Fixes flaky role test (#19899)
legrego 84f4a8d
Merge remote-tracking branch 'upstream/master' into security-app-privs
kobelb b464f03
Now with even easier repository access
kobelb e02c5bb
Sample was including login/version privileges, which was occasionally…
kobelb 9cdf641
Dynamic types (#19925)
kobelb a53e7d0
start to address feedback
legrego e11d86b
Merge remote-tracking branch 'upstream/master' into security-app-privs
kobelb f875cec
Fix RBAC Phase 1 merge from master (#20226)
legrego c899985
Merge branch 'master' into security-app-privs
legrego be6445c
Retrying initialize 20 times with a scaling backoff (#20297)
kobelb e3c1a99
Alternate legacy fallback (#20322)
kobelb a2cc325
Merge remote-tracking branch 'upstream/master' into security-app-privs
kobelb b8a110b
Setting the status to red on the first error then continually (#20343)
kobelb 8696030
Renaming get*Privilege to get*Action
kobelb a56af65
Adding "instance" to alert about other application privileges
kobelb 19a7d6f
Revising some of the naming for the edit roles screen
kobelb 33a153d
One more edit role variable renamed
kobelb 5d8745f
hasPrivileges is now checkPrivileges
kobelb c1689f7
Revising check_license tests
kobelb c7ae3e8
Adding 2 more privileges tests
kobelb 03f7931
Moving the other _find method to be near his friend
kobelb 98acdc0
Spelling "returning" correctly, whoops
kobelb 3a95af3
Adding Privileges tests
kobelb 3299a0a
tests for Elasticsearch's privileges APIs
legrego 6942034
Switching the hard-coded resource from 'default' to *
kobelb 19ddaea
Throw error before we execute a POST privilege call that won't work
kobelb 1f48041
Resolving issue when initially registering privileges
kobelb 98ea1b5
Logging legacy fallback deprecation warning on login (#20493)
kobelb 7cef606
Deriving application from Kibana index (#20614)
kobelb fb724d2
Validate ES has_privileges response before trusting it (#20682)
legrego c65908b
Removing unused setting
kobelb 3e6c057
Merge remote-tracking branch 'upstream/master' into security-app-privs
kobelb a597976
Public Role APIs (#20732)
kobelb 36f4b2f
Adding setting to allow the user to turn off the legacy fallback (#20…
kobelb 6fc19e6
Pulling the version from the kibana server
kobelb e170337
Deleting unused file
kobelb 95dbb99
Add API integration tests for roles with index and app privileges (#2…
kobelb 759330e
Rbac phase1 functional UI tests (#20949)
rashmivkulkarni beabf5d
Merge remote-tracking branch 'upstream/master' into security-app-privs
kobelb 40aab3f
Merge remote-tracking branch 'upstream/master' into security-app-privs
kobelb c33ab6c
Fixing role management API from users
kobelb f8645f3
Set a timeout when we try/catch a find, so it doesn't pause a long time
kobelb 3e248ba
Changing the way we detect if a user is reserved for the ftr
kobelb 066ec5b
Skipping flaky test
kobelb 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
There are no files selected for viewing
This file contains 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
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
/* | ||
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
* or more contributor license agreements. Licensed under the Elastic License; | ||
* you may not use this file except in compliance with the Elastic License. | ||
*/ | ||
|
||
export const ALL_RESOURCE = '*'; |
This file contains 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
This file contains 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
14 changes: 14 additions & 0 deletions
14
x-pack/plugins/security/public/services/application_privilege.js
This file contains 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,14 @@ | ||
/* | ||
* Copyright Elasticsearch B.V. and/or licensed to Elasticsearch B.V. under one | ||
* or more contributor license agreements. Licensed under the Elastic License; | ||
* you may not use this file except in compliance with the Elastic License. | ||
*/ | ||
|
||
import 'angular-resource'; | ||
import { uiModules } from 'ui/modules'; | ||
|
||
const module = uiModules.get('security', ['ngResource']); | ||
module.service('ApplicationPrivileges', ($resource, chrome) => { | ||
const baseUrl = chrome.addBasePath('/api/security/v1/privileges'); | ||
return $resource(baseUrl); | ||
}); |
This file contains 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
This file contains 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
This file contains 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.
Is 1024 the max length of the role names in Elasticsearch?
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.
Yup https://www.elastic.co/guide/en/x-pack/current/defining-roles.html