Current implementation uses the key name of the transfered asset as id for the secrets associated with the transfer. #2417
FlorianSeidelAWS
started this conversation in
Ideas
Replies: 2 comments 1 reply
-
Which ones are you specifically referring to? |
Beta Was this translation helpful? Give feedback.
0 replies
-
TransferProcessManager::storeProvisionedSecrets() stores secret using key name of provisioned resource |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
-
I noticed, that an assets key name is used for naming secrets used in the transfer process. Since there is no guarantee, that this is going to be unique in the future, this can cause problems. May I propose to replace this with the transfer process id? I think it is available in all places in the code where secrets are stored, deleted an retrieved for transfers.
Beta Was this translation helpful? Give feedback.
All reactions