🐛 Bug Report: [Server Side SDK] - Incorrect Return Value for Functions.getDeployment with Invalid Deployment ID

[ItzNotABug]

👟 Reproduction steps

When invoking the functions.getDeployment(...) method in the Server Side SDK with an empty deploymentId, the function unexpectedly returns a list of all deployments.

This behavior is similar to the functions.listDeployments(...) method.

const deploymentId = '';
const functionId = 'testFunction';
const deployment = await functions.getDeployment(functionId, deploymentId);

Above returns:

{
   "total": numberOfDeploymentsHere,
   "deployments":[
     { latestDeploymentInfo },
     { olderDeploymentInfo },
     { olderDeploymentInfo2 },
     { olderDeploymentInfo3 },
   ]
}

👍 Expected behavior

When the deploymentId is empty, the SDK should throw an AppwriteException instead of returning all deployments.

👎 Actual Behavior

The backend returns a list of all deployments/build.

🎲 Appwrite version

Different version (specify in environment)

💻 Operating system

Linux

🧱 Your Environment

Tested on : 1.3.8 & 1.4.2.

👀 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

[stnguyen90]

@ItzNotABug, thanks for raising this issue! 🙏🏼