chore(all): Delete logs that print password (#509)

NaverCloudPlatform · Jan 7, 2025 · 44bbb87 · 44bbb87
1 parent 58d3fcf
commit 44bbb87
Show file tree

Hide file tree

Showing 11 changed files with 1 addition and 24 deletions.
diff --git a/internal/service/cdss/cdss_cluster.go b/internal/service/cdss/cdss_cluster.go
@@ -241,7 +241,6 @@ func resourceNcloudCDSSClusterCreate(ctx context.Context, d *schema.ResourceData
 		ConfigGroupNo:            *GetInt32FromString(d.GetOk("config_group_no")),
 	}
 
-	LogCommonRequest("resourceNcloudCDSSClusterCreate", reqParams)
 	resp, _, err := config.Client.Vcdss.V1Api.ClusterCreateCDSSClusterReturnServiceGroupInstanceNoPost(ctx, reqParams)
 	if err != nil {
 		LogErrorResponse("resourceNcloudCDSSClusterCreate", err, reqParams)

diff --git a/internal/service/hadoop/hadoop.go b/internal/service/hadoop/hadoop.go
@@ -551,8 +551,6 @@ func (r *hadoopResource) Create(ctx context.Context, req resource.CreateRequest,
 		reqParams.UseDataCatalog = plan.UseDataCatalog.ValueBoolPointer()
 	}
 
-	tflog.Info(ctx, "CreateHadoop reqParams="+common.MarshalUncheckedString(reqParams))
-
 	response, err := r.config.Client.Vhadoop.V2Api.CreateCloudHadoopInstance(reqParams)
 	if err != nil {
 		resp.Diagnostics.AddError("CREATING ERROR", err.Error())

diff --git a/internal/service/mongodb/mongodb.go b/internal/service/mongodb/mongodb.go
@@ -568,8 +568,6 @@ func (m *mongodbResource) Create(ctx context.Context, req resource.CreateRequest
 		reqParams.DataStorageTypeCode = plan.DataStorageType.ValueStringPointer()
 	}
 
-	tflog.Info(ctx, "CreateMongoDb reqParams="+common.MarshalUncheckedString(reqParams))
-
 	response, err := m.config.Client.Vmongodb.V2Api.CreateCloudMongoDbInstance(reqParams)
 	if err != nil {
 		resp.Diagnostics.AddError("CREATING ERROR", err.Error())

diff --git a/internal/service/mssql/mssql.go b/internal/service/mssql/mssql.go
@@ -373,8 +373,6 @@ func (r *mssqlResource) Create(ctx context.Context, req resource.CreateRequest,
 		reqParams.BackupTime = plan.BackupTime.ValueStringPointer()
 	}
 
-	tflog.Info(ctx, "CreateMssql reqParams="+common.MarshalUncheckedString(reqParams))
-
 	response, err := r.config.Client.Vmssql.V2Api.CreateCloudMssqlInstance(reqParams)
 	if err != nil {
 		resp.Diagnostics.AddError("CREATING ERROR", err.Error())

diff --git a/internal/service/mysql/mysql.go b/internal/service/mysql/mysql.go
@@ -532,8 +532,6 @@ func (r *mysqlResource) Create(ctx context.Context, req resource.CreateRequest,
 		}
 	}
 
-	tflog.Info(ctx, "CreateMysql reqParams="+common.MarshalUncheckedString(reqParams))
-
 	response, err := r.config.Client.Vmysql.V2Api.CreateCloudMysqlInstance(reqParams)
 	if err != nil {
 		resp.Diagnostics.AddError("CREATING ERROR", err.Error())

diff --git a/internal/service/mysql/mysql_users.go b/internal/service/mysql/mysql_users.go
@@ -192,8 +192,6 @@ func (r *mysqlUsersResource) Create(ctx context.Context, req resource.CreateRequ
 		CloudMysqlUserList:   convertToCloudMysqlUserParameter(plan.MysqlUserList),
 	}
 
-	tflog.Info(ctx, "CreateMysqlUserList reqParams="+common.MarshalUncheckedString(reqParams))
-
 	response, err := r.config.Client.Vmysql.V2Api.AddCloudMysqlUserList(reqParams)
 	if err != nil {
 		resp.Diagnostics.AddError("CREATING ERROR", err.Error())
@@ -271,7 +269,6 @@ func (r *mysqlUsersResource) Update(ctx context.Context, req resource.UpdateRequ
 			CloudMysqlInstanceNo: state.ID.ValueStringPointer(),
 			CloudMysqlUserList:   convertToCloudMysqlUserParameter(plan.MysqlUserList),
 		}
-		tflog.Info(ctx, "ChangecloudMysqlUserList reqParams="+common.MarshalUncheckedString(reqParams))
 
 		response, err := r.config.Client.Vmysql.V2Api.ChangeCloudMysqlUserList(reqParams)
 		if err != nil {

diff --git a/internal/service/nasvolume/nas_volume.go b/internal/service/nasvolume/nas_volume.go
@@ -337,8 +337,6 @@ func createClassicNasVolume(d *schema.ResourceData, config *conn.ProviderConfig)
 		IsReturnProtection:              BoolPtrOrNil(d.GetOk("is_return_protection")),
 	}
 
-	LogCommonRequest("createClassicNasVolume", reqParams)
-
 	resp, err := config.Client.Server.V2Api.CreateNasVolumeInstance(reqParams)
 	if err != nil {
 		LogErrorResponse("createClassicNasVolume", err, reqParams)
@@ -364,8 +362,6 @@ func createVpcNasVolume(d *schema.ResourceData, config *conn.ProviderConfig) (*s
 		IsReturnProtection:              BoolPtrOrNil(d.GetOk("is_return_protection")),
 	}
 
-	LogCommonRequest("createVpcNasVolume", reqParams)
-
 	resp, err := config.Client.Vnas.V2Api.CreateNasVolumeInstance(reqParams)
 	if err != nil {
 		LogErrorResponse("createVpcNasVolume", err, reqParams)

diff --git a/internal/service/postgresql/postgresql.go b/internal/service/postgresql/postgresql.go
@@ -567,8 +567,6 @@ func (r *postgresqlResource) Create(ctx context.Context, req resource.CreateRequ
 		}
 	}
 
-	tflog.Info(ctx, "CreatePostgresql reqParams="+common.MarshalUncheckedString(reqParams))
-
 	response, err := r.config.Client.Vpostgresql.V2Api.CreateCloudPostgresqlInstance(reqParams)
 	if err != nil {
 		resp.Diagnostics.AddError("CREATING ERROR", err.Error())

diff --git a/internal/service/postgresql/postgresql_users.go b/internal/service/postgresql/postgresql_users.go
@@ -180,8 +180,6 @@ func (r *postgresqlUsersResource) Create(ctx context.Context, req resource.Creat
 		CloudPostgresqlUserList:   convertToCloudPostgresqlUserParameter(plan.PostgresqlUserList),
 	}
 
-	tflog.Info(ctx, "CreatePostgresqlUserList reqParams="+common.MarshalUncheckedString(reqParams))
-
 	response, err := r.config.Client.Vpostgresql.V2Api.AddCloudPostgresqlUserList(reqParams)
 	if err != nil {
 		resp.Diagnostics.AddError("CREATING ERROR", err.Error())
@@ -260,7 +258,6 @@ func (r *postgresqlUsersResource) Update(ctx context.Context, req resource.Updat
 			CloudPostgresqlInstanceNo: state.ID.ValueStringPointer(),
 			CloudPostgresqlUserList:   convertToCloudPostgresqlUserParameter(plan.PostgresqlUserList),
 		}
-		tflog.Info(ctx, "ChangeCloudPostgresqlUserList reqParams="+common.MarshalUncheckedString(reqParams))
 
 		response, err := r.config.Client.Vpostgresql.V2Api.ChangeCloudPostgresqlUserList(reqParams)
 		if err != nil {

diff --git a/internal/service/redis/redis.go b/internal/service/redis/redis.go
@@ -456,8 +456,6 @@ func (r *redisResource) Create(ctx context.Context, req resource.CreateRequest,
 		reqParams.CloudRedisUserPassword = plan.UserPassword.ValueStringPointer()
 	}
 
-	tflog.Info(ctx, "CreateCloudRedisInstance reqParams="+common.MarshalUncheckedString(reqParams))
-
 	response, err := r.config.Client.Vredis.V2Api.CreateCloudRedisInstance(reqParams)
 	if err != nil {
 		resp.Diagnostics.AddError("CREATING ERROR", err.Error())

diff --git a/internal/service/ses/ses_cluster.go b/internal/service/ses/ses_cluster.go
@@ -314,7 +314,7 @@ func resourceNcloudSESClusterCreate(ctx context.Context, d *schema.ResourceData,
 		MasterNodeCount:           masterNodeCount,
 		LoginKeyName:              StringPtrOrNil(d.GetOk("login_key_name")),
 	}
-	LogCommonRequest("resourceNcloudSESClusterCreate", reqParams)
+
 	resp, _, err := config.Client.Vses.V2Api.CreateClusterUsingPOST(ctx, *reqParams)
 	if err != nil {
 		LogErrorResponse("resourceNcloudSESClusterCreate", err, reqParams)