Move security api from plugins to clients (opensearch-project#442)

* fix(security): move security api from plugins to clients

Signed-off-by: florian <florian@harfanglab.fr>

* chore(CHANGELOG): update CHANGELOG for the PR#442

Signed-off-by: florian <florian@harfanglab.fr>

---------

Signed-off-by: florian <florian@harfanglab.fr>

CHANGELOG.md

@@ -19,6 +19,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/)
 ### Changed
 - Upgrading pytest-asyncio to latest version - 0.21.0 ([#339](https://github.com/opensearch-project/opensearch-py/pull/339))
 - Fixed flaky CI tests by replacing httpbin with a simple http_server ([#395](https://github.com/opensearch-project/opensearch-py/pull/395))
+- Move security from plugins to clients ([#442](https://github.com/opensearch-project/opensearch-py/pull/442))
 ### Deprecated
 ### Removed
 - Removed tests against Python 2.7 in github workflows ([#421](https://github.com/opensearch-project/opensearch-py/pull/421))

docs/source/api-ref/client.md

@@ -28,6 +28,7 @@ clients/ingest_client
 clients/indices_client
 clients/nodes_client
 clients/remote_client
+clients/security_client
 clients/snapshot_client
 clients/tasks_client
 clients/features_client

docs/source/api-ref/clients/security_client.md

@@ -0,0 +1,5 @@
+# Security Client
+
+```{eval-rst}
+.. autoclass:: opensearchpy.clients.security.SecurityClient
+```

docs/source/api-ref/plugins.md

@@ -9,5 +9,4 @@ maxdepth: 1
 
 plugins/alerting_plugin
 plugins/index_management_plugin
-plugins/security_plugin
 ```

opensearchpy/_async/client/__init__.py

@@ -39,6 +39,7 @@
 from .nodes import NodesClient
 from .plugins import PluginsClient
 from .remote import RemoteClient
+from .security import SecurityClient
 from .snapshot import SnapshotClient
 from .tasks import TasksClient
 from .utils import SKIP_IN_PATH, _bulk_body, _make_path, _normalize_hosts, query_params
@@ -196,12 +197,14 @@ class as kwargs, or a string in the format of ``host[:port]`` which will be
         self.ingest = IngestClient(self)
         self.nodes = NodesClient(self)
         self.remote = RemoteClient(self)
+        self.security = SecurityClient(self)
         self.snapshot = SnapshotClient(self)
         self.tasks = TasksClient(self)
-        self.plugins = PluginsClient(self)
 
         self.features = FeaturesClient(self)
 
+        self.plugins = PluginsClient(self)
+
     def __repr__(self):
         try:
             # get a list of all connections

opensearchpy/_async/client/__init__.pyi

@@ -39,6 +39,7 @@ from .indices import IndicesClient
 from .ingest import IngestClient
 from .nodes import NodesClient
 from .remote import RemoteClient
+from .security import SecurityClient
 from .snapshot import SnapshotClient
 from .tasks import TasksClient
 
@@ -54,6 +55,7 @@ class AsyncOpenSearch(object):
     ingest: IngestClient
     nodes: NodesClient
     remote: RemoteClient
+    security: SecurityClient
     snapshot: SnapshotClient
     tasks: TasksClient
     def __init__(

opensearchpy/_async/client/plugins.py

@@ -11,7 +11,6 @@
 
 from ..plugins.alerting import AlertingClient
 from ..plugins.index_management import IndexManagementClient
-from ..plugins.security import SecurityClient
 from .utils import NamespacedClient
 
 
@@ -25,7 +24,6 @@ def __init__(self, client):
         # self.anomaly_detection = AnomalyDetectionClient(client)
         # self.trace_analytics = TraceAnalyticsClient(client)
         self.index_management = IndexManagementClient(client)
-        self.security = SecurityClient(client)
 
         self._dynamic_lookup(client)
 
@@ -40,7 +38,6 @@ def _dynamic_lookup(self, client):
             # "anomaly_detection",
             # "trace_analytics",
             "index_management",
-            "security",
         ]
         for plugin in plugins:
             if not hasattr(client, plugin):

opensearchpy/_async/client/plugins.pyi

@@ -15,5 +15,4 @@ from .utils import NamespacedClient as NamespacedClient
 class PluginsClient(NamespacedClient):
     alerting: Any
     index_management: Any
-    security: Any
     def __init__(self, client: AsyncOpenSearch) -> None: ...

opensearchpy/client/__init__.py

@@ -40,6 +40,7 @@
 from .nodes import NodesClient
 from .plugins import PluginsClient
 from .remote import RemoteClient
+from .security import SecurityClient
 from .snapshot import SnapshotClient
 from .tasks import TasksClient
 from .utils import SKIP_IN_PATH, _bulk_body, _make_path, _normalize_hosts, query_params
@@ -197,6 +198,7 @@ class as kwargs, or a string in the format of ``host[:port]`` which will be
         self.ingest = IngestClient(self)
         self.nodes = NodesClient(self)
         self.remote = RemoteClient(self)
+        self.security = SecurityClient(self)
         self.snapshot = SnapshotClient(self)
         self.tasks = TasksClient(self)
 

opensearchpy/client/__init__.pyi

@@ -39,6 +39,7 @@ from .indices import IndicesClient
 from .ingest import IngestClient
 from .nodes import NodesClient
 from .remote import RemoteClient
+from .security import SecurityClient
 from .snapshot import SnapshotClient
 from .tasks import TasksClient
 
@@ -54,6 +55,7 @@ class OpenSearch(object):
     ingest: IngestClient
     nodes: NodesClient
     remote: RemoteClient
+    security: SecurityClient
     snapshot: SnapshotClient
     tasks: TasksClient
     def __init__(

opensearchpy/client/plugins.py

@@ -12,7 +12,6 @@
 
 from ..plugins.alerting import AlertingClient
 from ..plugins.index_management import IndexManagementClient
-from ..plugins.security import SecurityClient
 from .utils import NamespacedClient
 
 
@@ -26,7 +25,6 @@ def __init__(self, client):
         # self.anomaly_detection = AnomalyDetectionClient(client)
         # self.trace_analytics = TraceAnalyticsClient(client)
         self.index_management = IndexManagementClient(client)
-        self.security = SecurityClient(client)
 
         self._dynamic_lookup(client)
 
@@ -41,7 +39,6 @@ def _dynamic_lookup(self, client):
             # "anomaly_detection",
             # "trace_analytics",
             "index_management",
-            "security",
         ]
         for plugin in plugins:
             if not hasattr(client, plugin):

opensearchpy/client/plugins.pyi

@@ -15,5 +15,4 @@ from .utils import NamespacedClient as NamespacedClient
 class PluginsClient(NamespacedClient):
     alerting: Any
     index_management: Any
-    security: Any
     def __init__(self, client: OpenSearch) -> None: ...