aws / aws/aws-encryption-sdk-cli

CLI doesn't declare a syntax error for invalid --overwrite parameter

Open
#98 0 comments 0 reactions 0 assignees View on GitHub
bug
Dominant language
Python
Stars
82
Forks
42
PR merge metrics
No merged PRs in 30d

Description

The CLI has --no-overwrite and --overwrite-metadata parameters. It has no documented --overwrite parameter. This should be a syntax error.

Environment:
* Linux ip-172-31-39-147 4.9.58-18.55.amzn1.x86_64 #1 SMP Thu Nov 2 04:38:47 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux
* aws-encryption-sdk-cli/1.0.2.6 aws-encryption-sdk/1.3.2

```
aws-crypto --encrypt --input myData \
--master-keys key=$cmkArn provider='aws-kms' \
--output myEncryptedMessage \
--encryption-context purpose=test \
--metadata-output ~/metadata \
--overwrite
2017-11-17 16:32:08,975 - MainThread - aws_encryption_sdk_cli - WARNING - Overwriting existing target file because no action was specified otherwise: myEncryptedMessage

(ENV) [ec2-user@ip-172-31-39-147 test]$ echo $?
0
```

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.