Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

'git secret hide' without source files presents errors but reports success #154

Closed
joshrabinowitz opened this issue Apr 17, 2018 · 3 comments
Assignees
Labels

Comments

@joshrabinowitz
Copy link
Collaborator

joshrabinowitz commented Apr 17, 2018

  • Steps to replicate:

Set up a git-secret repo set up with a hidden config.ini (encrypted with git secret add config.ini and git secret hide -d config.ini)

% ls config.ini*
config.ini.secret

and you type git secret hide (no filenames)

  • Actual results: You get the output:
% git secret hide
sha256sum: /full/path/config.ini: No such file or directory  # <- note error
done. all 1 files are hidden.

And the file is not re-encrypted.

  • Expected results:

A better error message (i.e. 'abort: source file not found: /full/path/config.ini") and a non-zero exit code

@sobolevn
Copy link
Owner

Sorry, I am missing one thing in your description: is this file added to our keychain?
Or it is just a random file with the .secret extension?

@joshrabinowitz
Copy link
Collaborator Author

Assuming the file has been added to the keychain, and the source removed via git secret hide -r

@sobolevn
Copy link
Owner

So, that a bug. 😞

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants