Skip to content

Commit a9539ad

Browse files
Security fix: Restrict MCP gateway config file permissions to 0600 (#8024)
1 parent fa47552 commit a9539ad

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

pkg/awmg/gateway.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -440,8 +440,8 @@ func rewriteMCPConfigForGateway(configPath string, config *MCPGatewayServiceConf
440440
gatewayLog.Printf("Writing %d bytes to config file", len(data))
441441
fmt.Fprintln(os.Stderr, console.FormatInfoMessage(fmt.Sprintf("Writing %d bytes to config file", len(data))))
442442

443-
// Write back to file
444-
if err := os.WriteFile(configPath, data, 0644); err != nil {
443+
// Write back to file with restricted permissions (0600) since it contains sensitive API keys
444+
if err := os.WriteFile(configPath, data, 0600); err != nil {
445445
gatewayLog.Printf("Failed to write rewritten config: %v", err)
446446
fmt.Fprintln(os.Stderr, console.FormatErrorMessage(fmt.Sprintf("Failed to write rewritten config: %v", err)))
447447
return fmt.Errorf("failed to write rewritten config: %w", err)

0 commit comments

Comments
 (0)