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

the value is printed while doing a "get" #13

Open
hriprsd opened this issue Feb 5, 2021 · 2 comments
Open

the value is printed while doing a "get" #13

hriprsd opened this issue Feb 5, 2021 · 2 comments

Comments

@hriprsd
Copy link

hriprsd commented Feb 5, 2021

When I try to get a value from consul, the entire value is printed in the pipeline. i.e anyone can view it from the concourse-ui
Sample pipeline:

resources:

  • name: config
    type: consul-kv
    source:
    token: my-acl-token
    host: my-consul.com
    tls_cert: my-cert-string
    tls_key: my-cert-key-string
    key: my/key

jobs:

  • name: get-my-consul-key
    plan:
    • get: config

UI Output:
image

This enables anyone (even without access to consul) can view the KV stored in the concourse ui

@mdb
Copy link
Owner

mdb commented Feb 8, 2021

@hriprsd I believe this is a duplicate of issue #12, which I don't believe is a valid issue.

This enables anyone (even without access to consul) can view the KV stored in the concourse ui

☝️ Regarding this point, I believe this is not entirely accurate, IIUC. I believe it's more accurate to say this allows anyone with an RBAC role that grants them viewing access to view the K/V via the Concourse UI. Furthermore, this is also true of any other Concourse resource whose metadata is surfaced to Concourse.

As I asked in issue #12 ...

Are you suggesting that concourse-consul-kv-resource prints the values of the Consul k/v pairs it tracks? If so, that is expected, no? If the k/v pairs are secrets, I would think they should be stored in a proper secrets manager, such as Vault and not fetched directly via the concourse-consul-kv-resource. Or am I misinterpreting?

☝️ Does this seem reasonable? Or am I mistaken or misunderstanding your use-case?

@hriprsd
Copy link
Author

hriprsd commented Feb 8, 2021 via email

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

No branches or pull requests

2 participants