🐛 Bug Report: Server sdk for node deleteFile api delete storage bucket instead of file.

[usamashafiqs]

👟 Reproduction steps

Server sdk for node deleteFile api delete storage bucket if file ID had empty string as parameter.
It should throw exception if file ID had empty string.

try {
    const res = await SDK.storage.deleteFile('BucketID', '');
    console.log(res);
} catch (e) {
    if (e instanceof AppwriteException) {
        console.log(e.message);
    }
    console.log(e);
}

👍 Expected behavior

It should throw exception if file ID had empty string provided instead of file ID.

👎 Actual Behavior

Server sdk for node deleteFile api delete storage bucket if file ID had empty string as parameter.

🎲 Appwrite version

Version 1.2.x

💻 Operating system

Linux

🧱 Your Environment

No response

👀 Have you spent some time to check if this issue has been raised before?

• I checked and didn't find similar issue

🏢 Have you read the Code of Conduct?

• I have read the Code of Conduct

[probablyArth]

you should probably raise this issue on sdk-generator or sdk-node

[freefreeP]

Hi all this is my first contribution/PR,

I have worked on this and raised a PR. Please review the PR.

Thank you

[joeyouss]

Hi @freefreeP , thank you for raising the PR, our team will review it soon!

[stnguyen90]

Btw, this is a problem in utopia-php framework due to how were matching routes. This PR should fix this problem: utopia-php/http#92