Re: [PR] Add some more logging into boot.go to facilitate future environment debugging [beam]

Thu, 30 Apr 2026 08:05:22 -0700 


gemini-code-assist[bot] commented on code in PR #38342:
URL: https://github.com/apache/beam/pull/38342#discussion_r3168890724


##########
sdks/python/container/boot.go:
##########
@@ -183,12 +183,17 @@ func launchSDKProcess() error {
        if err != nil {
                logger.Fatalf(ctx, "Failed to convert pipeline options: %v", 
err)
        }
+       logger.Printf(ctx, "PipelineOptions=%v", options)

Review Comment:
   
![security-high](https://www.gstatic.com/codereviewagent/security-high-priority.svg)
 ![high](https://www.gstatic.com/codereviewagent/high-priority.svg)
   
   Logging the entire `PipelineOptions` object is a security risk as it may 
contain sensitive information such as API keys, access tokens, or passwords 
passed as pipeline arguments. It is recommended to avoid logging the full 
object or to implement a scrubbing mechanism to mask sensitive fields before 
logging.



-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to