Skip to content

Commit df9af61

Browse files
authored
fix(api): missing app secret for git checkout with https (#5833)
1 parent 70a24b0 commit df9af61

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

engine/api/workflow_run.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -1225,7 +1225,7 @@ func saveWorkflowRunSecrets(ctx context.Context, db *gorp.DbMap, projID int64, w
12251225
for _, v := range variables {
12261226
var wrSecret sdk.WorkflowRunSecret
12271227
switch {
1228-
case strings.HasPrefix(v.Name, "cds.app.") || strings.HasPrefix(v.Name, "cds.key."):
1228+
case strings.HasPrefix(v.Name, "cds.app.") || strings.HasPrefix(v.Name, "cds.key.") || v.Name == "git.http.password":
12291229
wrSecret = sdk.WorkflowRunSecret{
12301230
WorkflowRunID: wr.ID,
12311231
Context: fmt.Sprintf(workflow.SecretAppContext, id),

sdk/parameter.go

+1-1
Original file line numberDiff line numberDiff line change
@@ -300,7 +300,7 @@ func ParametersMerge(src []Parameter, overwritter []Parameter) []Parameter {
300300
return params
301301
}
302302

303-
// ParametersMapMerge merges two maps of parameters preserving all git values
303+
// ParametersMapMerge merges two maps of parameters
304304
func ParametersMapMerge(params map[string]string, otherParams map[string]string, checkFuncs ...func(string) bool) map[string]string {
305305
for k, overrideValue := range otherParams {
306306
if _, ok := params[k]; ok {

0 commit comments

Comments
 (0)