fix(deps): Update module github.com/baidubce/bce-sdk-go to v0.9.266 (main)#21754
fix(deps): Update module github.com/baidubce/bce-sdk-go to v0.9.266 (main)#21754renovate-sh-app[bot] wants to merge 1 commit intomainfrom
Conversation
There was a problem hiding this comment.
Cursor Bugbot has reviewed your changes and found 1 potential issue.
Bugbot Autofix prepared a fix for the issue found in the latest run.
- ✅ Fixed: Debug
fmt.Printfreplaces properlog.Warnflogging call- Replaced the stray fmt.Printf in GeneratePresignedUrlInternal with log.Warnf while preserving the existing fmt import for other uses.
Or push these changes by commenting:
@cursor push 6de68288f1
Preview (6de68288f1)
diff --git a/vendor/github.com/baidubce/bce-sdk-go/services/bos/api/object.go b/vendor/github.com/baidubce/bce-sdk-go/services/bos/api/object.go
--- a/vendor/github.com/baidubce/bce-sdk-go/services/bos/api/object.go
+++ b/vendor/github.com/baidubce/bce-sdk-go/services/bos/api/object.go
@@ -994,7 +994,7 @@
}
err := validateObjectKey(object)
if method == http.GET && err != nil {
- fmt.Printf("cannot generate presigned url: %v", err)
+ log.Warnf("cannot generate presigned url: %v", err)
return ""
}
req.SetMethod(method)You can send follow-ups to the cloud agent here.
Reviewed by Cursor Bugbot for commit 4fe854f. Configure here.
| log.Warnf("objectKey '%s' is invalid, cannot generate presigned url.", object) | ||
| err := validateObjectKey(object) | ||
| if method == http.GET && err != nil { | ||
| fmt.Printf("cannot generate presigned url: %v", err) |
There was a problem hiding this comment.
Debug fmt.Printf replaces proper log.Warnf logging call
Medium Severity
In GeneratePresignedUrlInternal, the object key validation error is logged via fmt.Printf instead of log.Warnf. The previous version used log.Warnf here, and the adjacent bucket validation on line 992 still correctly uses log.Warnf. This bypasses the structured logging framework, writes directly to stdout without a trailing newline, and won't appear in log aggregation systems. Looks like a leftover debug statement in the upstream SDK.
Reviewed by Cursor Bugbot for commit 4fe854f. Configure here.
3521f18 to
50efe5c
Compare
50efe5c to
85f2b65
Compare
9b7961c to
97506e5
Compare
| datasource | package | from | to | | ---------- | ------------------------------ | -------- | -------- | | go | github.com/baidubce/bce-sdk-go | v0.9.265 | v0.9.266 | Signed-off-by: renovate-sh-app[bot] <219655108+renovate-sh-app[bot]@users.noreply.github.com>
97506e5 to
c9b5a6c
Compare



This PR contains the following updates:
v0.9.265→v0.9.266Release Notes
baidubce/bce-sdk-go (github.com/baidubce/bce-sdk-go)
v0.9.266Compare Source
【版本:v0.9.266】
涉及产品: BOS
对象无标签时get object tagging应答200
bucket清单shedule取值增加一次性清单设置
预签名URL生成接口完善object key合法性检查
GetObjectMeta设置BceRestore字段
增加bucket name字符合法性检查逻辑
Configuration
📅 Schedule: (UTC)
🚦 Automerge: Enabled.
♻ Rebasing: Whenever PR is behind base branch, or you tick the rebase/retry checkbox.
🔕 Ignore: Close this PR and you won't be reminded about this update again.
Need help?
You can ask for more help in the following Slack channel: #proj-renovate-self-hosted. In that channel you can also find ADR and FAQ docs in the Resources section.