Fix update data source & block update endpint (opensearch-project#2364)

Signed-off-by: Kristen Tian <[email protected]> Signed-off-by: Kristen Tian <[email protected]>
kristenTian · Sep 15, 2022 · efb7c27 · efb7c27
1 parent 442111d
commit efb7c27
Show file tree

Hide file tree

Showing 3 changed files with 11 additions and 8 deletions.
diff --git a/src/plugins/data_source/server/saved_objects/data_source_saved_objects_client_wrapper.ts b/src/plugins/data_source/server/saved_objects/data_source_saved_objects_client_wrapper.ts
@@ -160,16 +160,19 @@ export class DataSourceSavedObjectsClientWrapper {
   }
 
   private async validateAndUpdatePartialAttributes<T = unknown>(attributes: T) {
-    const { auth } = attributes;
+    const { auth, endpoint } = attributes;
+
+    if (endpoint) {
+      throw SavedObjectsErrorHelpers.createBadRequestError(
+        `Update data source endpoint is not supported`
+      );
+    }
 
     if (auth === undefined) {
       return attributes;
     }
 
-    const {
-      type,
-      credentials: { password },
-    } = auth;
+    const { type, credentials } = auth;
 
     switch (type) {
       case AuthType.NoAuth:
@@ -179,7 +182,7 @@ export class DataSourceSavedObjectsClientWrapper {
           credentials: undefined,
         };
       case AuthType.UsernamePasswordType:
-        if (password) {
+        if (credentials && credentials.password) {
           return {
             ...attributes,
             auth: await this.encryptCredentials(auth),

diff --git a/...agement/public/components/edit_data_source/components/edit_form/edit_data_source_form.tsx b/...agement/public/components/edit_data_source/components/edit_form/edit_data_source_form.tsx
@@ -206,10 +206,10 @@ export class EditDataSourceForm extends React.Component<EditDataSourceProps, Edi
 
   onClickUpdateDataSource = () => {
     if (this.isFormValid()) {
+      // update data source endpoint is currently not supported/allowed
       const formValues: DataSourceAttributes = {
         title: this.state.title,
         description: this.state.description,
-        endpoint: this.props.existingDataSource.endpoint,
         auth: this.state.auth,
       };
       /* Remove credentials object for NoAuth */

diff --git a/src/plugins/data_source_management/public/types.ts b/src/plugins/data_source_management/public/types.ts
@@ -70,7 +70,7 @@ export const credentialSourceOptions = [
 export interface DataSourceAttributes extends SavedObjectAttributes {
   title: string;
   description?: string;
-  endpoint: string;
+  endpoint?: string;
   auth: {
     type: AuthType;
     credentials: UsernamePasswordTypedContent | undefined;