Skip to content
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.

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

RMB cache path is not compatible with Windows #1305

Open
scottyeager opened this issue Jan 9, 2025 · 1 comment
Open

RMB cache path is not compatible with Windows #1305

scottyeager opened this issue Jan 9, 2025 · 1 comment
Labels
rmb-sdk belongs to rmb
Milestone

Comments

@scottyeager
Copy link

The way we form this RMB cache path creates a path that's invalid for Windows:

path := filepath.Join(os.TempDir(), "rmb-cache", u.Host)

See here for an example of how this manifests.

What's happening is that the Host string includes a : and this is a reserved character in Windows that's not allowed in paths.

@scottyeager
Copy link
Author

I have tested and confirmed that just swapping any : for - in the path fixes this issue.

@rawdaGastan rawdaGastan added rmb-sdk belongs to rmb and removed rmb-sdk belongs to rmb type_bug Something isn't working labels Jan 26, 2025
@rawdaGastan rawdaGastan added this to the v0.17.x milestone Jan 26, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
rmb-sdk belongs to rmb
Projects
None yet
Development

No branches or pull requests

2 participants