Use `--local` for basic remote setup?
See original GitHub issueHi there, not sure if this is a bug or a feature, but the docs are suggesting to store .dvc/config
in git.
Which seemed fine at first, but then I noticed that my Azure Blob Storage account (i.e. my Azure username) is also stored in .dvc/config
, which means it would end up in git. Which is not ideal for team collaboration scenarios. What’s less ideal (read: really scary) is that connection strings entered using dvc remote modify blah connection_string ...
also end up in .dvc/config
, making them end up in git, making them end up in interesting places for open source projects.
Am I doing something obviously wrong? I wouldn’t expect the getting started docs to go very deep into security issues, but I wouldn’t expect them to store connection strings in source control either.
Issue Analytics
- State:
- Created 2 years ago
- Comments:16 (14 by maintainers)
Let me illustrate my case:
Assuming you don’t see the
--local
flag when quickly scanning the command reference (and not the command examples, which don’t include it), the first command example you’d encounter is if you felt the need to scroll really down in the Available parameteres per storage section. Which you may or may not do.@jorgeorpinel But following it specifically requires you to do a
dvc push
, which requires authenticating to a remote, which in turn will store the auth in the config. Not sure how this works for S3, but that’s what happens in Azure at least.@jorgeorpinel The first two samples, for url and account_name don’t mention --local. The other samples mention it, but you have to scroll for a screen to even see them.
To summarize my position, I wanted to point out what I see as a shortcoming of the docs, especially for beginners who don’t know any better - they will pay even less attention than I did to the fine details, and will end up pushing their connection strings to their closed or open repos.
I realize that different needs have to be balanced here, so feel free to handle this feedback in any way you see fit.
@jorgeorpinel yep, but we should control the number of things we mention (e.g. #2969 ). Goal of the get started was always simplicity first. I would consider in this case to improve this part
an actual remote and picking Azure from the same page
.