Threagile / threagile

Agile Threat Modeling Toolkit
https://threagile.io
MIT License
577 stars 126 forks source link

change single dash long arguments to double dash long arguments #64

Closed CMon closed 3 weeks ago

CMon commented 2 months ago

with threagile version > 0.9.0, a single dash for long arguments do not work anymore

ezavgorodniy commented 2 months ago

Could you please add more details on exact example when it's not working for you?

Also @joreiche may share more details on using confi file instead of arguments in command line - that's not documented but may improve a usability of the tool

CMon commented 2 months ago

sure:

$ docker run --rm -it -v "$(pwd)":/work threagile create-example-model -output /work 
unknown shorthand flag: 'o' in -output

and if i use the double dash:

$ docker run --rm -it -v "$(pwd)":/work threagile create-example-model --output /work

it works even if I also get the help printed as well, same for the -model and --model:

$ docker run --rm -it -v "$(pwd)":/work threagile analyze --output /work -model /work/threagile-example-model.yaml
unknown shorthand flag: 'm' in -model

and with double dash:

$ docker run --rm -it -v "$(pwd)":/work threagile analyze --output /work --model /work/threagile-example-model.yaml
failed to read and analyze model: unable to check risk tracking: Risk tracking references unknown risk (risk id not found) - you might want to use the option -ignore-orphaned-risk-tracking: unencrypted-asset@*

even if the example is not working anymore at least it started with the parsing

CMon commented 2 months ago

@ezavgorodniy is there anything else you need as information?

ezavgorodniy commented 3 weeks ago

Conflicts are resolved manually and changes are in master