Skip to content

fix(drivers/crypt): remove hard dependency on RemotePath #1713

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Merged
jyxjjj merged 15 commits into from
Nov 27, 2025
Merged

fix(drivers/crypt): remove hard dependency on RemotePath #1713

jyxjjj merged 15 commits into from
Nov 27, 2025

Conversation

@jyxjjj
Copy link
Member

@jyxjjj jyxjjj commented Nov 26, 2025 •
edited by j2rong4cn
Loading

Description

refuse to load crypt driver when upstream driver is not ready
return error when upstream driver is down
增加crypt驱动灵活性

Motivation and Context

Close #1702
Close #1445

How Has This Been Tested?

Checklist / 检查清单

  • I have read the CONTRIBUTING document.
  • I have formatted my code with go fmt or prettier.
  • I have added appropriate labels to this PR (or mentioned needed labels in the description if lacking permissions).
  • I have requested review from relevant code authors using the "Request review" feature when applicable.

KirCute
KirCute previously approved these changes Nov 26, 2025
View reviewed changes
@jyxjjj jyxjjj marked this pull request as draft November 26, 2025 06:10
@jyxjjj jyxjjj changed the title fix(drivers/crypt): refuse to load crypt driver when upstream driver is down fix(drivers/crypt): return error when upstream driver is down Nov 26, 2025
@jyxjjj jyxjjj marked this pull request as ready for review November 26, 2025 13:43
@jyxjjj jyxjjj requested a review from KirCute November 26, 2025 13:43
@j2rong4cn j2rong4cn changed the title fix(drivers/crypt): return error when upstream driver is down fix(drivers/crypt): remove hard dependency on RemotePath Nov 26, 2025
KirCute
KirCute requested changes Nov 26, 2025
View reviewed changes
@j2rong4cn j2rong4cn requested a review from KirCute November 27, 2025 11:10
@j2rong4cn j2rong4cn force-pushed the crypt-refuse-load branch 2 times, most recently from c2821a2 to 5ff0a79 Compare November 27, 2025 11:32
@jyxjjj jyxjjj merged commit c5f57bb into main Nov 27, 2025
12 checks passed
@jyxjjj jyxjjj deleted the crypt-refuse-load branch November 27, 2025 17:21
@BrewTestBot BrewTestBot mentioned this pull request Jan 1, 2026
Merged
lyy2005a2 added a commit to lyy2005a2/OpenList that referenced this pull request Jan 3, 2026
@lyy2005a2
修复（驱动程序/加密）：移除对远程路径（OpenListTeam#1713）的硬依赖)
d7c9f5b
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@KirCute KirCute KirCute approved these changes

@j2rong4cn j2rong4cn j2rong4cn left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

[BUG] crypt指向的目标路径失效时会导致重启，并且无法进入存储管理界面 建议加密驱动Crypt延迟启动

4 participants

@jyxjjj @KirCute @j2rong4cn