Skip to content

Warn on wildcard remote directories#420

Open
huangzesen wants to merge 1 commit into
spedas:masterfrom
huangzesen:fix/bleeding-edge-142-spd-download-wildcard-warning
Open

Warn on wildcard remote directories#420
huangzesen wants to merge 1 commit into
spedas:masterfrom
huangzesen:fix/bleeding-edge-142-spd-download-wildcard-warning

Conversation

@huangzesen

Copy link
Copy Markdown

Summary

  • warn when remote_path contains wildcard characters before URL expansion
  • warn when remote_file directory components contain wildcard characters
  • keep existing behavior unchanged: the warning is informational and downloads still proceed

Fixes #142

Validation

  • static check: warning block appears before url = remote_path + remote_file and before spd_download_expand
  • no IDL runtime was available here, so this was not runtime-reproduced locally

Review

  • Zhipu/GLM review gate: reviewed before PR; remote_path coverage added after review; no blocking findings

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.

spd_download should warn if wildcards are used in the remote directory name

1 participant