Merge pull request #214 from joe0BAB/fix/derive-command

joe0BAB · web-flow · commit d1517571d801 · 2025-04-25T16:13:56.000+02:00
fix(oauth): copy over all existing policies
diff --git a/src/extension/host-binary/cmd/main.go b/src/extension/host-binary/cmd/main.go
@@ -279,5 +279,5 @@ func runDeriveSecret(ctx context.Context, opts deriveOptions) error {
 	if !slices.Contains(s.Policies, mcpPolicyName) {
 		s.Policies = append(s.Policies, mcpPolicyName)
 	}
-	return c.SetSecret(ctx, secretsapi.Secret{Name: opts.Dst, Value: s.Value, Policies: []string{mcpPolicyName}})
+	return c.SetSecret(ctx, secretsapi.Secret{Name: opts.Dst, Value: s.Value, Policies: s.Policies})
 }

Original file line number	Diff line number	Diff line change
`@@ -279,5 +279,5 @@ func runDeriveSecret(ctx context.Context, opts deriveOptions) error {`
`279`	`279`	`if !slices.Contains(s.Policies, mcpPolicyName) {`
`280`	`280`	`s.Policies = append(s.Policies, mcpPolicyName)`
`281`	`281`	`}`
`282`		`- return c.SetSecret(ctx, secretsapi.Secret{Name: opts.Dst, Value: s.Value, Policies: []string{mcpPolicyName}})`
	`282`	`+ return c.SetSecret(ctx, secretsapi.Secret{Name: opts.Dst, Value: s.Value, Policies: s.Policies})`
`283`	`283`	`}`