mirror of
https://github.com/dutchcoders/transfer.sh.git
synced 2026-02-06 07:22:17 +00:00
Bug: Error 404 when try to delete file uploaded via zsh function #155
Reference in New Issue
Block a user
Delete Branch "%!s()"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Originally created by @slmnsh on GitHub.
First of it doesn't give any deletion token when we upload file via zsh function/plugin
Another thing is it shows delete button whenever someone visits the link and it prompts for deletion token but when we click on confirm button it shows "Error (404) during deletion of file"
So, I'll suggest to remove delete button completely. and also I want to know when files deleted from the server?
@paolafrancesca commented on GitHub:
We won't remove the delete button, since it's working properly: you have to provide the deletion token, as stated in the hint.
When you click the button a
DELETErequest is made totoken/filename/deletionToken, if the deletion token is wrong or empty, the request result in a 404 (not found)Deletion token is returned on upload in the header
X-Url-Delete, zsh function doesn't print it: I never used zsh so I wouldn't be able to modify. Any external PR to enanche the zsh function is welcome@slmnsh commented on GitHub:
So I think it's useless to show that delete button
On Wed, Sep 15, 2021, 1:51 AM Andrea Spacca @.***>
wrote:
@slmnsh commented on GitHub:
Where can I find the script for transfer cli command?
@slmnsh commented on GitHub:
Okay, I'll try that. Also when you click confirm again and again it appends the error into same error instead of replacing it completely.
@paolafrancesca commented on GitHub:
every server can setup it's own purge period.
If the server you've uploaded to is using
v1.3.0the purge time is reported in the homepage@corporateuser commented on GitHub:
Some command line clients do not print headers by default, please consider to return JSON object instead of a simple string:
@imTHAI commented on GitHub:
That's why ! I was searching where the hell I should be supposed to have received it. Maybe it would be good to warm people somewhere in the README.md text, zsh is even the default shell under MacOS
@paolafrancesca commented on GitHub:
@corporateuser sorry, that will be a breaking change
@slmnsh commented on GitHub:
nvm I made PR on transfer.sh-web repo for that fix