Skip to content

fix(baidu_netdisk): deplicate retry (redo #7972, link #8180)#8210

Merged
xhofe merged 1 commit intoAlistGo:mainfrom
ljcbaby:patch-2
Mar 27, 2025
Merged

fix(baidu_netdisk): deplicate retry (redo #7972, link #8180)#8210
xhofe merged 1 commit intoAlistGo:mainfrom
ljcbaby:patch-2

Conversation

@ljcbaby
Copy link
Copy Markdown
Contributor

@ljcbaby ljcbaby commented Mar 23, 2025

必须显示指定不重试,不然会用默认的重试参数

@KirCute
Copy link
Copy Markdown
Contributor

KirCute commented Mar 23, 2025

#8211 已经改成在线程内部获取信号量了,即使有重试也不会再 panic 了,要不还是改成原来那个 Attempts 3 次吧

@ljcbaby
Copy link
Copy Markdown
Contributor Author

ljcbaby commented Mar 23, 2025

#8211 已经改成在线程内部获取信号量了,即使有重试也不会再 panic 了,要不还是改成原来那个 Attempts 3 次吧

有两层重试,一层 RestyClient 一层 errgroup 这样看日志的时候报错会有点懵。

去掉一层逻辑和日志符合一点,或者得调整一下内外两层的日志输出。

@xhofe xhofe merged commit 7b62dcb into AlistGo:main Mar 27, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants