diff --git a/CHANGELOG.md b/CHANGELOG.md index 7d0f8170..4e47ec93 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -23,6 +23,7 @@ Inspired from [Keep a Changelog](https://keepachangelog.com/en/1.0.0/) - Fixes integration test response validation when response is null ([#572](https://github.com/opensearch-project/opensearch-go/pull/572)) - Adjust security Role struct for FLS from string to []string ([#572](https://github.com/opensearch-project/opensearch-go/pull/572)) - Fixes wrong response parsing for indices mapping and recovery ([#572](https://github.com/opensearch-project/opensearch-go/pull/572)) +- Fixes wrong response parsing for security get requests ([#572](https://github.com/opensearch-project/opensearch-go/pull/572)) ### Security diff --git a/plugins/security/api_actiongroups.go b/plugins/security/api_actiongroups.go index a5c5755a..28f31f5e 100644 --- a/plugins/security/api_actiongroups.go +++ b/plugins/security/api_actiongroups.go @@ -24,7 +24,7 @@ func (c actiongroupsClient) Get(ctx context.Context, req *ActionGroupsGetReq) (A data ActionGroupsGetResp err error ) - if data.response, err = c.apiClient.do(ctx, req, &data); err != nil { + if data.response, err = c.apiClient.do(ctx, req, &data.Groups); err != nil { return data, err } diff --git a/plugins/security/api_internalusers.go b/plugins/security/api_internalusers.go index 88399be0..5fe9f756 100644 --- a/plugins/security/api_internalusers.go +++ b/plugins/security/api_internalusers.go @@ -24,7 +24,7 @@ func (c internalusersClient) Get(ctx context.Context, req *InternalUsersGetReq) data InternalUsersGetResp err error ) - if data.response, err = c.apiClient.do(ctx, req, &data); err != nil { + if data.response, err = c.apiClient.do(ctx, req, &data.Users); err != nil { return data, err } diff --git a/plugins/security/api_roles.go b/plugins/security/api_roles.go index 8bfa5581..8a21c335 100644 --- a/plugins/security/api_roles.go +++ b/plugins/security/api_roles.go @@ -24,7 +24,7 @@ func (c rolesClient) Get(ctx context.Context, req *RolesGetReq) (RolesGetResp, e data RolesGetResp err error ) - if data.response, err = c.apiClient.do(ctx, req, &data); err != nil { + if data.response, err = c.apiClient.do(ctx, req, &data.Roles); err != nil { return data, err } diff --git a/plugins/security/api_rolesmapping.go b/plugins/security/api_rolesmapping.go index d3062a79..58341457 100644 --- a/plugins/security/api_rolesmapping.go +++ b/plugins/security/api_rolesmapping.go @@ -24,7 +24,7 @@ func (c rolesmappingClient) Get(ctx context.Context, req *RolesMappingGetReq) (R data RolesMappingGetResp err error ) - if data.response, err = c.apiClient.do(ctx, req, &data); err != nil { + if data.response, err = c.apiClient.do(ctx, req, &data.RolesMapping); err != nil { return data, err } diff --git a/plugins/security/api_tenants.go b/plugins/security/api_tenants.go index 2067b95a..a4d60a6a 100644 --- a/plugins/security/api_tenants.go +++ b/plugins/security/api_tenants.go @@ -24,7 +24,7 @@ func (c tenantsClient) Get(ctx context.Context, req *TenantsGetReq) (TenantsGetR data TenantsGetResp err error ) - if data.response, err = c.apiClient.do(ctx, req, &data); err != nil { + if data.response, err = c.apiClient.do(ctx, req, &data.Tenants); err != nil { return data, err }