I prefer to use TOML as credentials storage file because TOML looks simpler and less error-prone
would you consider adding support for it? I am willing to help to submit pr too
There is a good article explaining the reason why TOML is better than YAML
https://arp242.net/weblog/yaml_probably_not_so_great_after_all.html
TOML support is something I would love to have in SOPS. I don’t think I
personally have the time to implement it myself, but I could definitely
review a PR for it.
On Fri 27. Jul 2018 at 10:58, Alex Yeung notifications@github.com wrote:
I prefer to use TOML as credentials storage file because TOML looks
simpler and less error-prone
would you consider adding support for it? I am willing to help to submit
pr tooThere is a good article explaining the reason why TOML is better than YAML
https://arp242.net/weblog/yaml_probably_not_so_great_after_all.html—
You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
https://github.com/mozilla/sops/issues/369, or mute the thread
https://github.com/notifications/unsubscribe-auth/ACJ-V8osmuaQnkDaeM_W8I6XXQMQcbd2ks5uKtZKgaJpZM4VjJyT
.>
Thanks,
Adrian Utrilla
@autrilla do you think this might be a good feature request for a newbie to tackle?
@mdanzinger not sure, there's a few stores (YAML, JSON, dotenv..) you can use as examples so it might be okay, but I'm not really familiar with the status of Go TOML parser libraries. If you give it a shot and have some questions, I'm definitely here to help!
FWIW I think TOML support in Go is fairly robust. I use https://github.com/BurntSushi/toml for some projects but there are several others too.
Yeah, seems like Go has decent TOML support. I'm going to give it a go when I've got some spare time and see what I can come up with. I may very well get some stuff wrong though.
@autrilla Hey! So I gave this a quick attempt over the weekend, but I had some difficulties ensuring the branches maintained the correct order, as the TOML library uses a map to store the keys. Any ideas?
@autrilla Hey! So I gave this a quick attempt over the weekend, but I had some difficulties ensuring the branches maintained the correct order, as the TOML library uses a map to store the keys. Any ideas?
Yeah, we had this problem with YAML too, which is why now we use https://godoc.org/gopkg.in/yaml.v2#MapSlice, which preserves order. I'm not sure if any Go TOML libraries support this.
Most helpful comment
@autrilla Hey! So I gave this a quick attempt over the weekend, but I had some difficulties ensuring the branches maintained the correct order, as the TOML library uses a map to store the keys. Any ideas?