Skip to content

Modify path handling in web.rs, fix when ".tar" is file download#15

Merged
gdamjan merged 1 commit intogdamjan:mainfrom
wuooyun:patch-1
Nov 21, 2025
Merged

Modify path handling in web.rs, fix when ".tar" is file download#15
gdamjan merged 1 commit intogdamjan:mainfrom
wuooyun:patch-1

Conversation

@wuooyun
Copy link
Contributor

@wuooyun wuooyun commented Nov 12, 2025

when i down load .tar file not tar path,
occur this error " thread 'actix-rt|system:0|arbiter:0' (8195) panicked at src/web.rs:38:55:
called Result::unwrap() on an Err value: Os { code: 2, kind: NotFound, message: "No such file or directory" }"

Remove canonicalization of the file path, fix it.

Remove canonicalization of the file path. for fix download .tar files
@gdamjan
Copy link
Owner

gdamjan commented Nov 12, 2025

when i down load .tar file not tar path,

can you give an example?

@wuooyun
Copy link
Contributor Author

wuooyun commented Nov 12, 2025

2025-11-12 222619

@wuooyun
Copy link
Contributor Author

wuooyun commented Nov 12, 2025

this example "ip" not path , , "root/ip" use canonicalize() will return false because no this path

@gdamjan gdamjan merged commit d180302 into gdamjan:main Nov 21, 2025
1 of 2 checks passed
@gdamjan
Copy link
Owner

gdamjan commented Nov 21, 2025

Even I don't understand the logic of that code :)

merged now, but I'll probably rewrite that part altogether

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