Arvindh

### In Brief: Instead of using `client.RawRequest` of VAULT SDK , we should use `client.Logical().Write(...)` `client.Logical().Write(...)` was used by the VAULT CLI `client.RawRequest` should not used directly as mentioned in...

> Rename `internal/client` to `internal/clients`. Done

LGTM In Future it will nice if coap server have CID support https://www.linkedin.com/pulse/what-dtls-connection-id-mieszko-mieru%C5%84ski/ Reference code for ID https://github.dev/plgd-dev/go-coap/blob/master/examples/dtls/cid/server/main.go

@WashingtonKK In `users/service.go` repoerr are returned by service function , Could you please change to respective service error and handle them as you did for other service

For the change secret requets, If we send wrong password like below ``` curl --location --request PATCH 'http://localhost/users/secret' \ --header 'Content-Type: application/json' \ --header 'Accept: application/json' \ --header 'Authorization: Bearer...

Issue token doesn't return proper error **What were you trying to achieve?** Login to Magistrala with unknow / invalid user identity ```bash curl --location 'http://localhost/users/tokens/issue' \ --header 'Content-Type: application/json' \...

@WashingtonKK We should not allow a user to have mutiple relationship with domain. And we also should not allow SuperAdmin to create new relationship with domain . So i suggesting...

let me explain this here case here for removing users from `Domain` Example: Doman name: `domain_1` **domain_1 Administrator:** `user_101` `user_102` **domain_1 Editor:** `user_1` **domain_1 Viewer:** `user_2` **domain_1 Member:** `user_3` `user_1`...

> if user_1 (domain_1 Editor) tries to remove user_2, user_3 , user_101 (domain administrator) , should it fail or just ignore the admin users and remove the other users? >...

@mteodor There is similar proposal https://github.com/absmach/magistrala/issues/1672 ,