secrets have been fixed #7409
secrets have been fixed #7409
Conversation
|
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
|
⚪ ⚪
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
|
⚪ ⚪ ⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
|
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
| "sa_secret_name"_a = EncloseAndEscapeString(signer ? TStringBuilder{} << "f1_" << folderId << name : TString{}, '"'), | ||
| "login"_a = EncloseAndEscapeString(GetLogin(setting).GetOrElse({}), '"'), | ||
| "password_secret_name"_a = EncloseAndEscapeString("k2" + name, '"')); | ||
| "password_secret_name"_a = EncloseAndEscapeString(TStringBuilder{} << "f2_" << folderId << name, '"')); |
There was a problem hiding this comment.
Предлагаю после folderId еще вставить подчеркивание
| namespace { | ||
|
|
||
| TString MakeSecretKeyName(const TString& prefix, const TString& folderId, const TString& name) { | ||
| return TStringBuilder{} << prefix << "_" << folderId << "_" << name << Endl; |
|
⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
|
⚪ ⚪
🟢
*please be aware that the difference is based on comparing your commit and the last completed build from the post-commit, check comparation |
2 participants
Changelog entry
...
Changelog category
Additional information
...