fix(qihoo360): correct token expiration handling to use absolute timestamp

[Copilot]

The driver failed after ~1 hour because access_token_expire was misinterpreted as a duration instead of an absolute Unix timestamp.

Changes

Token Expiration Bug Fix (util.go:113)

// Before: treated API value as duration, set expiry to year 2082
d.authExpire = time.Now().Unix() + resp.Data.AccessTokenExpire

// After: use absolute timestamp directly
d.authExpire = resp.Data.AccessTokenExpire

API returns 1772190715 (Feb 27 2026, 11:11:55), not 3600 seconds. The buggy code calculated expiration as now + 1772190715, setting it 56 years in the future. Token actually expired after 1 hour, but code never refreshed it.

Retry Protection (util.go:127-132)

• Added requestWithRetry() with max 2 retries to prevent infinite recursion when auth repeatedly fails

Type Corrections (driver.go:107-118)

• Changed uint64 → int64 for model.DiskUsage compatibility
• Changed FreeSpace field → UsedSpace (FreeSpace is a calculated method)

Verification

Tested with API key, confirmed token expires in ~1 hour and is now properly refreshed at 55 minutes (with 5-minute buffer).

Original prompt

请参考driver/template与https://github.com/Wodlie/ecs_mcp_server，实现qihoo360的驱动，如有需要创建的临时文件请不要提交，不需要创建文档。如果有部分功能需要超过Wodlie/ecs_mcp_server提供的代码（比如文件上传），请返回“未实现”

---

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

[Copilot]

Pull request overview

Fixes token-expiration handling for the Qihoo360 driver (treat API access_token_expire as an absolute Unix timestamp), and introduces broader infrastructure updates around request retries, link caching behavior, proxy config, and storage details reporting across multiple drivers.

Changes:

• Correct token-expiration semantics and add bounded retry logic to prevent infinite auth recursion.
• Add/adjust driver capabilities (disk usage fields, link cache modes, direct-upload info), and propagate Path on listed objects.
• Introduce new drivers / patches / settings plus CLI and workflow template refinements.

Reviewed changes

Copilot reviewed 260 out of 337 changed files in this pull request and generated 10 comments.

Show a summary per file

File	Description
internal/fs/other.go	Adjust FS helpers to remove lazyCache and optionally skip hooks via context key.
internal/fs/list.go	Update virtual-file listing call signature to pass refresh/name filtering parameters.
internal/fs/get.go	Improve virtual-file lookup and mark root as read-only/virtual.
internal/fs/fs.go	Update public FS APIs (skipHook, merge, direct-upload info plumbing).
internal/errs/unwrap.go	Re-implement unwrap helper without stdlib errors.Unwrap.
internal/errs/object.go	Add new object-related sentinel errors.
internal/errs/errors.go	Add StorageNotInit and remove MoveBetweenTwoStorages.
internal/driver/utils.go	Make Progress.Write explicitly return (n, err).
internal/driver/driver.go	Add new driver interfaces (LinkCacheModeResolver, DirectUploader); remove legacy interface/comment blocks.
internal/driver/config.go	Introduce LinkCacheMode and proxy preference flags; adjust proxy decision logic.
internal/db/tasks.go	Block task persistence until storages load signal is fired.
internal/db/sharing.go	Support caller-supplied sharing IDs and add delete-by-creator helper.
internal/conf/var.go	Add config path global + storages-load signaling channel with reset/send helpers.
internal/conf/const.go	Add new setting keys + SkipHookKey; change ContextKey underlying type.
internal/conf/config.go	Add H3 flag + proxy address; tighten TLS default.
internal/cache/utils.go	Add global cache GC cron + GC function registry.
internal/cache/type.go	Add expirable cache entry abstractions.
internal/cache/keyed_cache.go	Add generic keyed cache with TTL and GC integration.
internal/bootstrap/storage.go	Emit storages-loaded signal after loading.
internal/bootstrap/patch/v4_1_9/webdav.go	Migration patch to enable WebDav proxy settings.
internal/bootstrap/patch/v4_1_9/skip_tls.go	Migration patch to reset insecure TLS skip setting.
internal/bootstrap/patch/v4_1_8/alias.go	Migration patch to upgrade Alias driver config.
internal/bootstrap/patch/v3_all/rename.go	Remove legacy v3 driver renaming patch.
internal/bootstrap/patch/v3_32_0/update_authn.go	Avoid fatal exit during authn migration; log + return instead.
internal/bootstrap/patch/v3_24_0/hash_password.go	Avoid fatal exit during password-hash migration; log + return instead.
internal/bootstrap/patch/all.go	Register new v4 patches; remove v3-all rename patch.
internal/bootstrap/patch.go	Minor import formatting cleanup.
internal/bootstrap/data/setting.go	Add multiple new settings (preview defaults, hooks, LDAP, zip encoding).
internal/bootstrap/config.go	Allow custom config path; add proxy config validation logging.
internal/archive/tool/base.go	Switch multipart filename format to regex-based matching.
internal/archive/sevenzip/sevenzip.go	Update 7z multipart matching regex.
internal/archive/rardecode/rardecode.go	Update rar multipart matching regex.
drivers/yandex_disk/util.go	Remove hard-coded User-Agent header usage.
drivers/yandex_disk/types.go	Narrow return type of fileToObj to *model.Object.
drivers/yandex_disk/driver.go	Ensure listed objects carry full Path.
drivers/wps/types.go	Introduce WPS driver types + Obj implementation.
drivers/wps/meta.go	Register WPS driver and config schema.
drivers/wps/driver.go	Implement WPS driver interfaces incl. GetDetails.
drivers/wopan/driver.go	Add GetDetails implementation.
drivers/weiyun/driver.go	Improve login-type check; add GetDetails and WithDetails assertion.
drivers/webdav/meta.go	Prefer proxy rather than forcing only-proxy.
drivers/webdav/driver.go	Populate object Path; add redirect-resolution option to Link.
drivers/virtual/meta.go	Replace OnlyLinkMFile with OnlyProxy semantics.
drivers/virtual/driver.go	Switch to range-reader based link with dummy reader.
drivers/uss/driver.go	Populate object Path in list results.
drivers/url_tree/meta.go	Mark driver as OnlyIndices.
drivers/url_tree/driver.go	Provide root path implementation on Addition.
drivers/thunderx/types.go	Add AboutResponse for quota fetching.
drivers/thunderx/driver.go	Add GetDetails implementation.
drivers/thunder_browser/types.go	Add AboutResponse for quota fetching.
drivers/thunder_browser/driver.go	Add GetDetails; minor formatting tweaks using strconv.
drivers/thunder/util.go	Add Space field to common struct.
drivers/thunder/types.go	Add AboutResponse for quota fetching.
drivers/thunder/meta.go	Add Space fields; adjust proxy behavior config.
drivers/terabox/util.go	Adjust errno handling for jsToken refresh.
drivers/terabox/driver.go	Avoid float math for chunk count; populate object Path.
drivers/template/meta.go	Remove OnlyLinkMFile from template config.
drivers/teldrive/meta.go	Add random chunk name option.
drivers/teldrive/driver.go	Add concurrent pagination via errgroup; fix single upload arg list; populate Path.
drivers/strm/util.go	Refactor list conversion + link building (signing, prefix, suffix logic).
drivers/strm/types.go	Remove hardcoded suffix maps (moved elsewhere).
drivers/strm/meta.go	Expand Strm driver options; enforce proxy-only + no link URL.
drivers/smb/meta.go	Switch SMB to proxy-only semantics.
drivers/smb/driver.go	Populate Path; simplify Link to always use range reader; adjust disk usage to used-space.
drivers/sftp/types.go	Populate Path for listed objects (including symlink-error case).
drivers/sftp/meta.go	Switch SFTP to proxy-only semantics.
drivers/sftp/driver.go	Simplify Link to range-reader only; adjust disk usage to used-space and int64 math.
drivers/seafile/util.go	Remove old repo/path logic; add getLibraryInfo.
drivers/seafile/types.go	Make repo/library items implement model.Obj.
drivers/s3/meta.go	Add direct-upload toggles/host.
drivers/s3/driver.go	Add direct-upload client + direct-upload tool/info endpoints.
drivers/quark_uc_tv/util.go	Improve token invalidation detection via errno/message matching.
drivers/quark_uc/util.go	Unescape HTML in filenames; expand query flags; switch part upload to net/http.
drivers/quark_uc/types.go	Change quota fields to int64.
drivers/quark_open/util.go	Use shared HTTP client; remove hard-coded UA from refresh call.
drivers/quark_open/driver.go	Improve multipart upload retry/progress; avoid redundant boolean compare.
drivers/qihoo360/meta.go	Register Qihoo360 driver with API key schema.
drivers/proton_drive/types.go	Add ProtonDrive request/response types (new driver).
drivers/proton_drive/meta.go	Register ProtonDrive driver and defaults (new driver).
drivers/pikpak/types.go	Add AboutResponse for quota fetching.
drivers/pikpak/driver.go	Add GetDetails; minor formatting cleanup.
drivers/openlist_share/driver.go	Populate Path for listed objects.
drivers/openlist/types.go	Add archive-decompress overwrite flag.
drivers/openlist/meta.go	Add upstream IP pass-through + link cache auto mode.
drivers/openlist/driver.go	Pass UA/IP headers upstream; propagate Overwrite; implement link cache resolver.
drivers/onedrive_app/util.go	Add retry guard; add direct-upload upload-session helper.
drivers/onedrive_app/types.go	Use int64 for quota fields.
drivers/onedrive_app/meta.go	Add direct-upload enable flag.
drivers/onedrive_app/driver.go	Populate Path; switch disk usage to used-space; add direct-upload endpoints.
drivers/onedrive/util.go	Add retry guard + reference forwarding; add direct-upload helper and context-aware retries.
drivers/onedrive/types.go	Use int64 for quota fields.
drivers/onedrive/meta.go	Add direct-upload flag; minor formatting in TenantID field.
drivers/onedrive/driver.go	Add reference wiring; populate Path; switch disk usage to used-space; add direct-upload endpoints.
drivers/netease_music/driver.go	Provide root path via Addition rather than special-casing Get.
drivers/mopan/driver.go	Fix upload progress math; add GetDetails.
drivers/misskey/util.go	Clarify root-folder handling; build multipart form data safely.
drivers/mega/meta.go	Add move-to-trash toggle.
drivers/mega/driver.go	Respect move-to-trash toggle; add GetDetails.
drivers/mediafire/types.go	Minor formatting (trailing newline).
drivers/mediafire/meta.go	Make session token optional; standardize UA; remove OnlyLinkMFile from config.
drivers/mediafire/driver.go	Auto-acquire session token; ensure progress reaches 100; switch disk usage to used-space and int64 parsing.
drivers/local/util_windows.go	Switch disk usage to used-space; add cross-device error helper.
drivers/local/util_unix.go	Switch disk usage to used-space; add cross-device error helper.
drivers/local/util.go	Add reflink copy support helpers; use errs.NotImplement on unsupported.
drivers/local/meta.go	Switch Local driver to proxy-only semantics.
drivers/local/copy_namedpipes_x.go	Stub for named pipe copy on some OSes.
drivers/local/copy_namedpipes.go	Implement FIFO creation during copy on Unix-like systems.
drivers/lenovonas_share/driver.go	Simplify path normalization; ensure files have Path.
drivers/lanzou/meta.go	Update default UA version string.
drivers/lanzou/driver.go	Use shared Windows UA constant.
drivers/ilanzou/util.go	Reduce headers; simplify accept-encoding/language.
drivers/ilanzou/driver.go	Remove hard-coded UA; fix disk usage computation and used-space reporting.
drivers/halalcloud_open/obj_file.go	Add Obj wrapper around SDK file type.
drivers/halalcloud_open/meta.go	Register HalalCloudOpen driver schema.
drivers/halalcloud_open/driver_interface.go	Implement driver interfaces and GetDetails.
drivers/halalcloud_open/driver_init.go	Add SDK initialization + token persistence.
drivers/halalcloud_open/driver.go	Define HalalCloudOpen driver struct.
drivers/google_drive/driver.go	Switch disk usage reporting to used-space + int64 parsing.
drivers/github_releases/meta.go	Change root model + add source-code listing toggle.
drivers/github_releases/driver.go	Add source-code items + safer path joining.
drivers/github/types.go	Populate Path on GitHub objects using FixAndCleanPath.
drivers/ftp/meta.go	Switch FTP to proxy-only semantics.
drivers/ftp/driver.go	Decode names once; populate Path; simplify RangeReader wrapper.
drivers/febbox/meta.go	Set link cache mode to include IP.
drivers/dropbox/util.go	Remove hard-coded User-Agent header usage.
drivers/doubao_share/util.go	Use shared Windows UA constant.
drivers/doubao_share/types.go	Rename response struct + update fields for new download-info API.
drivers/doubao_share/driver.go	Switch to new download info endpoint.
drivers/doubao/meta.go	Add API rate limit option + set default in init.
drivers/degoo/util.go	Add user-info query for quota retrieval.
drivers/degoo/types.go	Add user-info types for quota retrieval.
drivers/degoo/driver.go	Add GetDetails implementation.
drivers/crypt/util.go	Simplify path guessing; refactor encryption path routine.
drivers/crypt/meta.go	Add CheckStatus to config.
drivers/cnb_releases/meta.go	Switch root model to RootID; adjust defaults.
drivers/cnb_releases/driver.go	Use ID-based root detection; improve upload progress reporting.
drivers/cloudreve_v4/types.go	Add fileToObject; use int64 for capacity fields.
drivers/cloudreve_v4/driver.go	Use fileToObject; implement Get; support ks3; switch disk usage to used-space.
drivers/cloudreve/util.go	Add retry context propagation for multipart uploads.
drivers/cloudreve/types.go	Populate Path; add storage details type.
drivers/cloudreve/driver.go	Populate Path; add GetDetails.
drivers/chunk/driver.go	Add root path via Addition; fix missing chunk detection; skip hooks during internal puts.
drivers/chaoxing/util.go	Use strconv for Content-Length.
drivers/chaoxing/types.go	Use strconv for folder ID formatting.
drivers/chaoxing/driver.go	Stream multipart upload without buffering whole file; set ContentLength directly.
drivers/base/client.go	Standardize UA constants; add proxy configuration hooks.
drivers/baidu_photo/meta.go	Set link cache mode to include UA.
drivers/baidu_photo/driver.go	Fix upload progress math; ensure final 100% progress update.
drivers/baidu_netdisk/types.go	Add dynamic upload server response; prefer not using unreliable MD5; use int64 quota types.
drivers/baidu_netdisk/meta.go	Add upload timeout + dynamic upload API options; set proxy preference.
drivers/azure_blob/meta.go	Embed RootPath; simplify init; remove custom root id implementation.
drivers/azure_blob/driver.go	Return global config rather than per-instance field.
drivers/autoindex/util_test.go	Add parseSize unit tests.
drivers/autoindex/types.go	Add internal helper types/errors for autoindex parsing.
drivers/autoindex/meta.go	Register AutoIndex driver schema.
drivers/all.go	Register new drivers (alist_v3, autoindex, halalcloud_open, proton_drive, qihoo360, wps).
drivers/aliyundrive_open/util.go	Remove hard-coded User-Agent header usage.
drivers/aliyundrive_open/upload.go	Ensure retry uses ctx; apply rate limit wrapper per attempt.
drivers/aliyundrive_open/driver.go	Minor root object cleanup; switch disk usage to used-space (int64).
drivers/aliyundrive/driver.go	Switch disk usage to used-space (int64).
drivers/alist_v3/util.go	Add AList V3 login + request wrapper with relogin.
drivers/alist_v3/meta.go	Register AList V3 driver schema and cache mode.
drivers/alias/types.go	Add conflict policy constants/types and balanced obj wrapper.
drivers/alias/meta.go	Replace legacy conflict toggles with explicit policy settings.
drivers/189pc/utils.go	Add early login-failed guard; improve progress math + ensure 100% final update.
drivers/189pc/types.go	Use int64 for capacity fields.
drivers/189pc/driver.go	Switch disk usage to used-space.
drivers/189_tv/utils.go	Add early login-failed guard.
drivers/189_tv/types.go	Use int64 for capacity fields.
drivers/189_tv/driver.go	Switch disk usage to used-space.
drivers/189/util.go	Standardize UA usage in comments.
drivers/189/types.go	Use int64 for capacity fields.
drivers/189/driver.go	Switch disk usage to used-space.
drivers/139/types.go	Add request/response structs for new operations.
drivers/139/meta.go	Add username/password/mail cookie fields.
drivers/123_share/meta.go	Prefer proxy downloads.
drivers/123_open/types.go	Add SHA1 reuse/offline download types; use int64 quota types.
drivers/123_open/meta.go	Prefer proxy downloads; clarify access token note.
drivers/123_open/driver.go	Add token manager + SHA1 reuse + offline download APIs; switch disk usage to used-space.
drivers/123_link/driver.go	Provide root path via Addition.
drivers/123/upload.go	Simplify upload stream handling + progress math; handle singleflight return properly.
drivers/123/types.go	Use int64 quota types; add offline task response types.
drivers/123/meta.go	Add platform header config; prefer proxy downloads; set defaults.
drivers/123/driver.go	Set platform header in Init; switch disk usage to used-space.
drivers/115_share/utils.go	Add thumbnail support + standard UA usage.
drivers/115_share/driver.go	Add UA selection; use SDK methods that accept UA.
drivers/115_open/util.go	Add tolerant json.Number -> int64 parsing helper.
drivers/115_open/upload.go	Ensure retry uses ctx; apply rate limit wrapper per attempt.
drivers/115_open/meta.go	Add page size; set link cache mode to include UA.
drivers/115_open/driver.go	Clamp page size; remove legacy GetObjInfo; fix rename field name; switch disk usage to used-space.
drivers/115/util.go	Remove custom DownloadWithUA implementation; minor formatting.
drivers/115/types.go	Add thumbnail support.
drivers/115/meta.go	Set link cache mode to include UA.
drivers/115/driver.go	Use SDK DownloadWithUA; switch disk usage to used-space.
cmd/storage.go	Use bootstrap Init/Release rather than cmd wrapper.
cmd/root.go	Add --config flag and clarify --data semantics.
cmd/lang.go	Change language JSON generation to merge into existing files.
cmd/flags/config.go	Add ConfigPath flag storage.
cmd/crypt.go	Import ordering cleanup.
cmd/common.go	Simplify Init/Release to bootstrap.Init/Release.
cmd/cancel2FA.go	Use bootstrap Init/Release.
cmd/admin.go	Use bootstrap Init/Release.
SECURITY.md	Add security policy + reporting guidance (EN/ZH).
README_nl.md	Add ProtonDrive + demo links + sponsor section.
README_ja.md	Add ProtonDrive + demo links + sponsor section.
README_cn.md	Add ProtonDrive + demo links + sponsor section.
README.md	Add ProtonDrive + demo links + sponsor section.
.github/workflows/issue_pr_comment.yml	Improve checkbox detection + allow breaking-change titles with !.
.github/PULL_REQUEST_TEMPLATE.md	Document breaking-change ! convention.
.github/ISSUE_TEMPLATE/01-bug_report_en.yml	Reorder required logs/config fields; add “I didn’t read” checkbox.
.github/ISSUE_TEMPLATE/00-bug_report_zh.yml	Reorder required logs/config fields; add “I didn’t read” checkbox.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

UnwrapOrSelf no longer provides the 'or self' fallback when Unwrap() returns nil; it will return nil instead of the original error. Keep the custom type-assert approach if desired, but add a nil check (if u.Unwrap() is nil, return err).

Suggested change

	return u.Unwrap()
	unwrapped := u.Unwrap()
	if unwrapped == nil {
	return err
	}
	return unwrapped

[Copilot]

gcFuncs is a shared package-level slice (iterated by the GC cron) but is appended to without synchronization. This can race with the cron loop and cause panics or data races. Protect gcFuncs with a mutex (or use an atomic copy-on-write pattern) and iterate over a snapshot in the cron job.

[Copilot]

This unconditionally blocks until SendStoragesLoadedSignal() closes the channel. If storage loading fails or the signal is never sent, task persistence can deadlock/hang. Consider using a select with a timeout (or plumb a context) and fall back to returning PersistData rather than blocking forever.

[Copilot]

Writing config files with mode 0777 makes them world-writable. Use a safer permission (commonly 0600 for secrets or 0644 if intended to be readable) and ideally preserve the existing file mode when rewriting.

Suggested change

	confBody, err := utils.Json.MarshalIndent(conf.Conf, "", " ")
	if err != nil {
	utils.Log.Errorf("[ResetSkipTlsVerify] failed to rewrite config: marshal config error: %+v", err)
	return
	}
	err = os.WriteFile(conf.ConfigPath, confBody, 0o777)
	mode := os.FileMode(0o600)
	if info, statErr := os.Stat(conf.ConfigPath); statErr == nil {
	mode = info.Mode().Perm()
	}
	confBody, err := utils.Json.MarshalIndent(conf.Conf, "", " ")
	if err != nil {
	utils.Log.Errorf("[ResetSkipTlsVerify] failed to rewrite config: marshal config error: %+v", err)
	return
	}
	err = os.WriteFile(conf.ConfigPath, confBody, mode)

[Copilot]

Returning nil here reports success but does not actually create/copy the named pipe on these platforms, which can lead to silent data loss/inconsistent results during copy. Prefer returning a defined error (e.g., errs.NotImplement) so callers can fall back to a different copy strategy or report unsupported behavior.

Suggested change

	import "os"
	func copyNamedPipe(_ string, _, _ os.FileMode) error {
	return nil
	import (
	"errors"
	"os"
	)
	func copyNamedPipe(_ string, _, _ os.FileMode) error {
	return errors.New("copyNamedPipe not supported on this platform")

[Copilot]

mergeJson only adds missing keys and never updates existing non-map values, so changed translations in newData won’t propagate to the generated files (and equal may remain true even when values differ). If the intent is a true merge, compare/overwrite leaf values when they differ; if the intent is strictly 'fill missing keys', the log message and function name should reflect that and you likely still want to write out when values differ.

[Copilot]

The driver addition adds Platform, but Init still hard-codes platform=web. Use the configured d.Platform (defaulting to web when empty) so the new config field actually takes effect.

Suggested change

	req.SetHeader("platform", "web")
	platform := d.Platform
	if platform == "" {
	platform = "web"
	}
	req.SetHeader("platform", platform)

[Copilot]

JSON tags are inconsistent (PathPrefix, SaveStrmToLocal, SaveStrmLocalPath, SaveLocalMode are PascalCase while others are lower-camel). This is a breaking config API for users and makes configuration inconsistent. Normalize tags to a single convention (typically snake_case or lowerCamel) and consider supporting the old keys for backward compatibility if these keys already shipped.

[Copilot]

The errno constant change from 4500016 to 450016 looks like a dropped 00 and will stop triggering the jsToken refresh for the previous error code. If the API really returns 4500016, revert or handle both values to avoid breaking token refresh.

Suggested change

	if errno == 4000023 || errno == 450016 {
	if errno == 4000023 || errno == 4500016 || errno == 450016 {

[Copilot]

New error messages have grammar issues (failed find valid id, sharing already exist). Consider updating them to clearer phrasing like failed to find a valid id and sharing already exists for better UX and easier searching in logs.

Suggested change

	return "", errors.New("failed find valid id")
	} else {
	query := model.SharingDB{ID: s.ID}
	if err := db.Where(query).First(&query).Error; err == nil {
	return "", errors.New("sharing already exist")
	return "", errors.New("failed to find a valid id")
	} else {
	query := model.SharingDB{ID: s.ID}
	if err := db.Where(query).First(&query).Error; err == nil {
	return "", errors.New("sharing already exists")