1230165 - taskcluster-secrets: service broken - fix to work with new auth endpoints

Status: RESOLVED FIXED

(Taskcluster :: Services, defect)

Description

[Pete Moore [:pmoore][:pete]]

Currently taskcluster-secrets is using the old no-longer-supported auth endpoint:

  * GET https://auth.taskcluster.net/v1/client/<clientId>/credentials

This has been superseded, see docs:

  * http://docs.taskcluster.net/auth/api-docs

This means the service no longer works. No doubt we'll need to upgrade dependencies. I tried upgrading to taskcluster-lib 0.8.12 but that didn't do the trick - maybe that requires the new config format? I'm not sure. Anyway, for now I'll put secrets in the worker type until we have taskcluster-secrets working again.

Note: this service lacks a CI as far as I can see - we should probably pimp it a bit with more detailed docs (e.g. link to docs site from README.md), get running in travis, maybe auto-release from travis to npm on git tag, explain how the service can be used to share secrets between tasks and dependent tasks, add some README badges, test coverage report, etc. Also explain in the project how to create a config file if you want to run tests... Then also maybe some information about how to restart the service etc if there are problems. Oh and maybe add some monitoring. Rather a lot still to do.

Oh - and maybe a front end on tools.taskcluster.net so people can query secrets or publish them via a web interface.

But for the first part, let's just get it working again.

Comment 1

[Dustin J. Mitchell [:dustin] (he/him)]

I just filed 1234630 to make a BZ component for this.  I'm the new maintainer, but I don't follow general so I didn't see this :(

I'll get some more bugs filed for basic stuff once we get the component set up.

Comment 2

[Dustin J. Mitchell [:dustin] (he/him)]

Hopefully
  https://github.com/taskcluster/taskcluster-secrets/commit/af0b283779b1389d99adfc7b0f86b27dfb142fed
should be sufficient

Comment 3

[Dustin J. Mitchell [:dustin] (he/him)]

Seems so.  The rest will end up in other bugs.