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.

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

[Rename] security-oss plugin #146

Merged
merged 2 commits into from
Mar 11, 2021

Conversation

kavilla
Copy link
Member

@kavilla kavilla commented Mar 10, 2021

Signed-off-by: Kawika Avilla [email protected]

Issue #, if available:
#37

Description of changes:
Rename files and variables under plugins/security-oss.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

Signed-off-by: Kawika Avilla <[email protected]>
Copy link
Contributor

@mihirsoni mihirsoni left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@boktorbb boktorbb left a comment

Choose a reason for hiding this comment

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

LGTM

@boktorbb boktorbb merged commit d39cd77 into opensearch-project:dashboards Mar 11, 2021
@kavilla kavilla deleted the rename-security-oss branch March 18, 2021 08:01
kavilla referenced this pull request in kavilla/OpenSearch-Dashboards Mar 19, 2021
* [Rename] security-oss plugin

Signed-off-by: Kawika Avilla <[email protected]>

* Missed a save. Will squash this commit

Signed-off-by: Kawika Avilla <[email protected]>
kavilla added a commit that referenced this pull request Mar 20, 2021
* [Rename] security-oss plugin

Signed-off-by: Kawika Avilla <[email protected]>

* Missed a save. Will squash this commit

Signed-off-by: Kawika Avilla <[email protected]>
mihirsoni pushed a commit that referenced this pull request Mar 20, 2021
* [Rename] security-oss plugin

Signed-off-by: Kawika Avilla <[email protected]>

* Missed a save. Will squash this commit

Signed-off-by: Kawika Avilla <[email protected]>
wanglam pushed a commit to wanglam/OpenSearch-Dashboards that referenced this pull request Sep 15, 2023
…ct#146)

* [Workspace] Add acl related functions for workspace

Signed-off-by: gaobinlong <[email protected]>

* Minor change

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>
gaobinlong added a commit to gaobinlong/OpenSearch-Dashboards that referenced this pull request Sep 20, 2023
…ct#146)

* [Workspace] Add acl related functions for workspace

Signed-off-by: gaobinlong <[email protected]>

* Minor change

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>
gaobinlong added a commit to gaobinlong/OpenSearch-Dashboards that referenced this pull request Sep 21, 2023
…ct#146)

* [Workspace] Add acl related functions for workspace

Signed-off-by: gaobinlong <[email protected]>

* Minor change

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>
SuZhou-Joe pushed a commit to SuZhou-Joe/OpenSearch-Dashboards that referenced this pull request Sep 21, 2023
…ct#146)

* [Workspace] Add acl related functions for workspace

Signed-off-by: gaobinlong <[email protected]>

* Minor change

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>
SuZhou-Joe pushed a commit to SuZhou-Joe/OpenSearch-Dashboards that referenced this pull request Sep 21, 2023
…ct#146)

* [Workspace] Add acl related functions for workspace

Signed-off-by: gaobinlong <[email protected]>

* Minor change

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>
wanglam pushed a commit to wanglam/OpenSearch-Dashboards that referenced this pull request Feb 22, 2024
…ct#146)

* [Workspace] Add acl related functions for workspace

Signed-off-by: gaobinlong <[email protected]>

* Minor change

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>
ruanyl pushed a commit that referenced this pull request Mar 5, 2024
* [Workspace] Add ACL related functions for workspace (#146)

* [Workspace] Add acl related functions for workspace

Signed-off-by: gaobinlong <[email protected]>

* Minor change

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>

* Modify changelog

Signed-off-by: gaobinlong <[email protected]>

* Add more unit test cases

Signed-off-by: gaobinlong <[email protected]>

* Modify test case

Signed-off-by: gaobinlong <[email protected]>

* Some minor change

Signed-off-by: gaobinlong <[email protected]>

* Add more test cases

Signed-off-by: gaobinlong <[email protected]>

* Optimize some code and the comments of the functions

Signed-off-by: gaobinlong <[email protected]>

* Add more comments for some basic functions

Signed-off-by: gaobinlong <[email protected]>

* Export more interfaces

Signed-off-by: gaobinlong <[email protected]>

* consume permissions in repository

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: consume permissions in serializer

Signed-off-by: SuZhou-Joe <[email protected]>

* Add unit tests for consuming permissions in repository

Signed-off-by: gaobinlong <[email protected]>

* Remove double exclamation

Signed-off-by: gaobinlong <[email protected]>

* Rename some variables

Signed-off-by: gaobinlong <[email protected]>

* Remove duplicated semicolon

Signed-off-by: gaobinlong <[email protected]>

* Add permissions field to the mapping only if the permission control is enabled

Signed-off-by: gaobinlong <[email protected]>

* Fix test failure

Signed-off-by: gaobinlong <[email protected]>

* Add feature flag config to the yml file

Signed-off-by: gaobinlong <[email protected]>

* Make the comment of feature flag more clear

Signed-off-by: gaobinlong <[email protected]>

* Make comment more clear

Signed-off-by: gaobinlong <[email protected]>

* Remove management permission type

Signed-off-by: gaobinlong <[email protected]>

* Fix test failure

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Co-authored-by: SuZhou-Joe <[email protected]>
gaobinlong added a commit to gaobinlong/OpenSearch-Dashboards that referenced this pull request Mar 5, 2024
* [Workspace] Add ACL related functions for workspace (opensearch-project#146)

* [Workspace] Add acl related functions for workspace

Signed-off-by: gaobinlong <[email protected]>

* Minor change

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>

* Modify changelog

Signed-off-by: gaobinlong <[email protected]>

* Add more unit test cases

Signed-off-by: gaobinlong <[email protected]>

* Modify test case

Signed-off-by: gaobinlong <[email protected]>

* Some minor change

Signed-off-by: gaobinlong <[email protected]>

* Add more test cases

Signed-off-by: gaobinlong <[email protected]>

* Optimize some code and the comments of the functions

Signed-off-by: gaobinlong <[email protected]>

* Add more comments for some basic functions

Signed-off-by: gaobinlong <[email protected]>

* Export more interfaces

Signed-off-by: gaobinlong <[email protected]>

* consume permissions in repository

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: consume permissions in serializer

Signed-off-by: SuZhou-Joe <[email protected]>

* Add unit tests for consuming permissions in repository

Signed-off-by: gaobinlong <[email protected]>

* Remove double exclamation

Signed-off-by: gaobinlong <[email protected]>

* Rename some variables

Signed-off-by: gaobinlong <[email protected]>

* Remove duplicated semicolon

Signed-off-by: gaobinlong <[email protected]>

* Add permissions field to the mapping only if the permission control is enabled

Signed-off-by: gaobinlong <[email protected]>

* Fix test failure

Signed-off-by: gaobinlong <[email protected]>

* Add feature flag config to the yml file

Signed-off-by: gaobinlong <[email protected]>

* Make the comment of feature flag more clear

Signed-off-by: gaobinlong <[email protected]>

* Make comment more clear

Signed-off-by: gaobinlong <[email protected]>

* Remove management permission type

Signed-off-by: gaobinlong <[email protected]>

* Fix test failure

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Co-authored-by: SuZhou-Joe <[email protected]>
gaobinlong added a commit to gaobinlong/OpenSearch-Dashboards that referenced this pull request Mar 8, 2024
* [Workspace] Add ACL related functions for workspace (opensearch-project#146)

* [Workspace] Add acl related functions for workspace

Signed-off-by: gaobinlong <[email protected]>

* Minor change

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>

* Modify changelog

Signed-off-by: gaobinlong <[email protected]>

* Add more unit test cases

Signed-off-by: gaobinlong <[email protected]>

* Modify test case

Signed-off-by: gaobinlong <[email protected]>

* Some minor change

Signed-off-by: gaobinlong <[email protected]>

* Add more test cases

Signed-off-by: gaobinlong <[email protected]>

* Optimize some code and the comments of the functions

Signed-off-by: gaobinlong <[email protected]>

* Add more comments for some basic functions

Signed-off-by: gaobinlong <[email protected]>

* Export more interfaces

Signed-off-by: gaobinlong <[email protected]>

* consume permissions in repository

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: consume permissions in serializer

Signed-off-by: SuZhou-Joe <[email protected]>

* Add unit tests for consuming permissions in repository

Signed-off-by: gaobinlong <[email protected]>

* Remove double exclamation

Signed-off-by: gaobinlong <[email protected]>

* Rename some variables

Signed-off-by: gaobinlong <[email protected]>

* Remove duplicated semicolon

Signed-off-by: gaobinlong <[email protected]>

* Add permissions field to the mapping only if the permission control is enabled

Signed-off-by: gaobinlong <[email protected]>

* Fix test failure

Signed-off-by: gaobinlong <[email protected]>

* Add feature flag config to the yml file

Signed-off-by: gaobinlong <[email protected]>

* Make the comment of feature flag more clear

Signed-off-by: gaobinlong <[email protected]>

* Make comment more clear

Signed-off-by: gaobinlong <[email protected]>

* Remove management permission type

Signed-off-by: gaobinlong <[email protected]>

* Fix test failure

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Co-authored-by: SuZhou-Joe <[email protected]>
Hailong-am pushed a commit to Hailong-am/OpenSearch-Dashboards that referenced this pull request Mar 8, 2024
…) (opensearch-project#287)

* [Workspace] Add ACL related functions for workspace (opensearch-project#146)

* [Workspace] Add acl related functions for workspace



* Minor change



---------



* Modify changelog



* Add more unit test cases



* Modify test case



* Some minor change



* Add more test cases



* Optimize some code and the comments of the functions



* Add more comments for some basic functions



* Export more interfaces



* consume permissions in repository



* feat: consume permissions in serializer



* Add unit tests for consuming permissions in repository



* Remove double exclamation



* Rename some variables



* Remove duplicated semicolon



* Add permissions field to the mapping only if the permission control is enabled



* Fix test failure



* Add feature flag config to the yml file



* Make the comment of feature flag more clear



* Make comment more clear



* Remove management permission type



* Fix test failure



---------

Signed-off-by: gaobinlong <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Co-authored-by: SuZhou-Joe <[email protected]>
yubonluo pushed a commit to yubonluo/OpenSearch-Dashboards that referenced this pull request Mar 12, 2024
…roject#279)

* [Workspace][Feature] Add ACL related functions (opensearch-project#5084)

* [Workspace] Add ACL related functions for workspace (opensearch-project#146)

* [Workspace] Add acl related functions for workspace

Signed-off-by: gaobinlong <[email protected]>

* Minor change

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>

* Modify changelog

Signed-off-by: gaobinlong <[email protected]>

* Add more unit test cases

Signed-off-by: gaobinlong <[email protected]>

* Modify test case

Signed-off-by: gaobinlong <[email protected]>

* Some minor change

Signed-off-by: gaobinlong <[email protected]>

* Add more test cases

Signed-off-by: gaobinlong <[email protected]>

* Optimize some code and the comments of the functions

Signed-off-by: gaobinlong <[email protected]>

* Add more comments for some basic functions

Signed-off-by: gaobinlong <[email protected]>

* Export more interfaces

Signed-off-by: gaobinlong <[email protected]>

* consume permissions in repository

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: consume permissions in serializer

Signed-off-by: SuZhou-Joe <[email protected]>

* Add unit tests for consuming permissions in repository

Signed-off-by: gaobinlong <[email protected]>

* Remove double exclamation

Signed-off-by: gaobinlong <[email protected]>

* Rename some variables

Signed-off-by: gaobinlong <[email protected]>

* Remove duplicated semicolon

Signed-off-by: gaobinlong <[email protected]>

* Add permissions field to the mapping only if the permission control is enabled

Signed-off-by: gaobinlong <[email protected]>

* Fix test failure

Signed-off-by: gaobinlong <[email protected]>

* Add feature flag config to the yml file

Signed-off-by: gaobinlong <[email protected]>

* Make the comment of feature flag more clear

Signed-off-by: gaobinlong <[email protected]>

* Make comment more clear

Signed-off-by: gaobinlong <[email protected]>

* Remove management permission type

Signed-off-by: gaobinlong <[email protected]>

* Fix test failure

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Co-authored-by: SuZhou-Joe <[email protected]>

* Remove duplicated attribute

Signed-off-by: gaobinlong <[email protected]>

* Enable acl in integration tests for workspace client wrapper

Signed-off-by: gaobinlong <[email protected]>

* Format the code

Signed-off-by: gaobinlong <[email protected]>

* Modify feature flag

Signed-off-by: gaobinlong <[email protected]>

* Modify feature flag to false

Signed-off-by: gaobinlong <[email protected]>

* Modify feature flag to true

Signed-off-by: gaobinlong <[email protected]>

* Enhance the test case

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Co-authored-by: SuZhou-Joe <[email protected]>
SuZhou-Joe pushed a commit to SuZhou-Joe/OpenSearch-Dashboards that referenced this pull request Mar 18, 2024
…ct#146)

* [Workspace] Add acl related functions for workspace

Signed-off-by: gaobinlong <[email protected]>

* Minor change

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>
SuZhou-Joe added a commit to SuZhou-Joe/OpenSearch-Dashboards that referenced this pull request Mar 18, 2024
…) (opensearch-project#287)

* [Workspace] Add ACL related functions for workspace (opensearch-project#146)

* [Workspace] Add acl related functions for workspace



* Minor change



---------



* Modify changelog



* Add more unit test cases



* Modify test case



* Some minor change



* Add more test cases



* Optimize some code and the comments of the functions



* Add more comments for some basic functions



* Export more interfaces



* consume permissions in repository



* feat: consume permissions in serializer



* Add unit tests for consuming permissions in repository



* Remove double exclamation



* Rename some variables



* Remove duplicated semicolon



* Add permissions field to the mapping only if the permission control is enabled



* Fix test failure



* Add feature flag config to the yml file



* Make the comment of feature flag more clear



* Make comment more clear



* Remove management permission type



* Fix test failure



---------

Signed-off-by: gaobinlong <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Co-authored-by: SuZhou-Joe <[email protected]>
opensearch-trigger-bot bot pushed a commit that referenced this pull request Apr 11, 2024
* [Workspace] Add ACL related functions for workspace (#146)

* [Workspace] Add acl related functions for workspace

Signed-off-by: gaobinlong <[email protected]>

* Minor change

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>

* Modify changelog

Signed-off-by: gaobinlong <[email protected]>

* Add more unit test cases

Signed-off-by: gaobinlong <[email protected]>

* Modify test case

Signed-off-by: gaobinlong <[email protected]>

* Some minor change

Signed-off-by: gaobinlong <[email protected]>

* Add more test cases

Signed-off-by: gaobinlong <[email protected]>

* Optimize some code and the comments of the functions

Signed-off-by: gaobinlong <[email protected]>

* Add more comments for some basic functions

Signed-off-by: gaobinlong <[email protected]>

* Export more interfaces

Signed-off-by: gaobinlong <[email protected]>

* consume permissions in repository

Signed-off-by: SuZhou-Joe <[email protected]>

* feat: consume permissions in serializer

Signed-off-by: SuZhou-Joe <[email protected]>

* Add unit tests for consuming permissions in repository

Signed-off-by: gaobinlong <[email protected]>

* Remove double exclamation

Signed-off-by: gaobinlong <[email protected]>

* Rename some variables

Signed-off-by: gaobinlong <[email protected]>

* Remove duplicated semicolon

Signed-off-by: gaobinlong <[email protected]>

* Add permissions field to the mapping only if the permission control is enabled

Signed-off-by: gaobinlong <[email protected]>

* Fix test failure

Signed-off-by: gaobinlong <[email protected]>

* Add feature flag config to the yml file

Signed-off-by: gaobinlong <[email protected]>

* Make the comment of feature flag more clear

Signed-off-by: gaobinlong <[email protected]>

* Make comment more clear

Signed-off-by: gaobinlong <[email protected]>

* Remove management permission type

Signed-off-by: gaobinlong <[email protected]>

* Fix test failure

Signed-off-by: gaobinlong <[email protected]>

---------

Signed-off-by: gaobinlong <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Co-authored-by: Josh Romero <[email protected]>
Co-authored-by: SuZhou-Joe <[email protected]>
(cherry picked from commit 54c36fe)
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
ananzh pushed a commit that referenced this pull request Apr 12, 2024
* [Workspace] Add ACL related functions for workspace (#146)

* consume permissions in repository
* feat: consume permissions in serializer
* Add unit tests for consuming permissions in repository
* Remove double exclamation
* Rename some variables
* Remove duplicated semicolon
* Add permissions field to the mapping only if the permission control is enabled
* Fix test failure
* Add feature flag config to the yml file
* Make the comment of feature flag more clear
* Make comment more clear
* Remove management permission type
* Fix test failure

---------

(cherry picked from commit 54c36fe)

Signed-off-by: gaobinlong <[email protected]>
Signed-off-by: SuZhou-Joe <[email protected]>
Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Co-authored-by: Josh Romero <[email protected]>
Co-authored-by: SuZhou-Joe <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants