ci-reporter icon indicating copy to clipboard operation
ci-reporter copied to clipboard

Work for private repos

Open JasonEtco opened this issue 6 years ago • 2 comments

Currently, ci-reporter will not work for private repositories because the CI providers that private repos use require an API key for private builds. Here are the things that need to happen for private repo support to be possible:

  • [ ] A UI (or some other method) to encrypt API keys
  • [ ] Store encrypted API keys in the .github/ci-reporter.yml file
  • [ ] Use those keys in API calls to CI providers

To be clear, I do not want to resort to using a database for these keys. There is too much overhead involved, and I'd like to nail down a good pattern for encrypting/storing keys in repositories since it'll have implications for many more apps.

JasonEtco avatar Mar 17 '18 22:03 JasonEtco

I'm all for this, it doesn't need to be complicated either. Need it for CodeBuild. Is anyone planning on doing this?

jpike88 avatar May 17 '18 05:05 jpike88

What would be an appropriate key to use to encrypt these secrets - the private key that the GitHub app has?

I investigated if Circle has a tidy way to generate an API key with a limited set of permissions but it does not so that would need to be communicated to the user, likely during the install & redirect phase of application install.

I think short of GitHub providing a store for secret material the best we can do is your prescribed approach.

What's the consensus between:

A) showing the user the encrypted key and asking them to commit it to config.yml

B) granting the application repository write permissions to abstract away the additional install step outlined in A?

Dombo avatar Dec 03 '18 09:12 Dombo