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

Add some helpful tips to the README #360

Merged
merged 2 commits into from
Jul 12, 2023
Merged

Conversation

anoadragon453
Copy link
Contributor

A couple notes that tripped me up when initially trying this. I hope they can help make future journeys smoother :)

  • Getting the syntax of .sops.yaml wrong can cause vague errors when it comes to encrypting secrets files.
  • path_regex needs to include all file extensions that you intend to encrypt. sops supports YAML, JSON, ENV, INI and binary files, so suggesting that those be picked up by default seems sensible. Personally, I had trouble figuring out why sops wouldn't accept my .env file - again with a confusing error message.

A couple notes that tripped me up when initially trying this. I hope they can help make future journeys smoother :)

- Getting the syntax of `.sops.yaml` wrong can cause vague errors when it comes to encrypting secrets files.
- `path_regex` needs to include all file extensions that you intend to encrypt. `sops` supports YAML, JSON, ENV, INI and binary files, so suggesting that those be picked up by default seems sensible. Personally, I had trouble figuring out why `sops` wouldn't accept my `.env` file - again with a confusing error message.
README.md Outdated Show resolved Hide resolved
@anoadragon453 anoadragon453 requested a review from Mic92 July 11, 2023 18:23
@Mic92
Copy link
Owner

Mic92 commented Jul 12, 2023

Thanks for the documentation contributions!
@mergify queue

@Mic92 Mic92 added the merge-queue merge after ci succeeds label Jul 12, 2023
@Mic92 Mic92 merged commit 88b964d into Mic92:master Jul 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
merge-queue merge after ci succeeds
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants