Feature request: Automatically convert analysis IDs to lowercase

Consensys / mythx-cli

A command line interface for the MythX smart contract security analysis API

https://mythx-cli.readthedocs.io/

MIT License

84 stars 29 forks source link

Feature request: Automatically convert analysis IDs to lowercase #94

Closed muellerberndt closed 4 years ago

muellerberndt commented 4 years ago

MythX CLI version: 0.6.5
Python version: 3.7.4
Operating System: Mac

Description

Convert analysis IDs to lowercase automatically. The dashboard shows IDs uppercase which means copy/pasting them into a "render" command doesn't work:

mythx render --markdown 0D07D850-7114-40C2-A544-8CF09E77DA84
Fetching report for analysis 0D07D850-7114-40C2-A544-8CF09E77DA84
The API returned an error:
Got unexpected status code 404: {"status":404,"error":"Analysis not found"}

dmuhs commented 4 years ago

Fixed in https://github.com/dmuhs/mythx-cli/pull/100. Now all render targets are converted to lowercase automatically before querying the API :+1: