Skip to content

fix(rclonecli): improve vfs cache refresh/forget logic and parameter types#649

Merged
javi11 merged 4 commits into
javi11:mainfrom
drondeseries:fix/rclonecli-vfs-refresh
Jun 3, 2026
Merged

fix(rclonecli): improve vfs cache refresh/forget logic and parameter types#649
javi11 merged 4 commits into
javi11:mainfrom
drondeseries:fix/rclonecli-vfs-refresh

Conversation

@drondeseries

Copy link
Copy Markdown
Contributor

Problem

The rclone VFS cache refresh/forget calls had two issues:

  1. Wrong parameter types — some rc call parameters were passed as string where the rclone RC API expects other types, causing silent failures or ignored cache refreshes
  2. Incorrect refresh vs forget logic — the code was calling vfs/refresh in cases where vfs/forget is the correct operation (and vice versa), meaning newly imported files were sometimes not appearing in the mount, or stale entries were not being evicted

Fix

  • Corrected parameter types for vfs/refresh and vfs/forget RC calls
  • Clarified when to use refresh (populate new dir entries) vs forget (evict stale cache) based on the operation context
  • Improved error handling and logging for VFS RC calls

Impact

Newly imported files now reliably appear under the rclone mount without requiring manual cache refreshes or container restarts.

@javi11 javi11 merged commit 8234773 into javi11:main Jun 3, 2026
2 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.

2 participants