denoland / deployctl

Command line tool for Deno Deploy
https://deno.com/deploy
MIT License
352 stars 58 forks source link

fix: use correct `deploymentId` for `logs --prod` #335

Open shiftinv opened 1 month ago

shiftinv commented 1 month ago

Fixes #275.

Just hit this issue myself earlier, and figured I'd look into it. c: productionDeployment.id is the build(?) UUID, whereas .deploymentId is the 12-char ID the endpoint seemingly expects here. .deployment.id also exists, but since they're identical and both are used across the codebase, I assume either one is fine.

CLAassistant commented 1 month ago

CLA assistant check
All committers have signed the CLA.