Write authorization #43

Merged
merged 5 commits into from Apr 10, 2013

Projects

None yet

2 participants

@robyoung

Authorization token for write API

jabley and others added some commits Apr 7, 2013
@jabley jabley Initial draft of write authorization
Needs more tests and work around deployment / revocation,  but
contains the general idea.
c597267
@jabley jabley Add missing thing
Running the tests was giving me warnings about this not being
available.
e5451b7
@robyoung robyoung Add headers when using HTTPTestClient
So headers such as the bearer token are passed onto the apis.

@maxfliri
@robyoung
38b6903
@robyoung robyoung Add reptiles bucket to test bearer tokens 96fdcc0
@robyoung robyoung Move tokens out into their own config files
This better reflects how they are deployed. Tokens change per
organisation but general configuration does not.

@maxfliri
@robyoung
e93abba
@jabley

LGTM :trollface:

Are there any changes required to deployment configuration for other environments?

@robyoung

Yes, good point, linked below. Basically, we have a shared configuration and then organisation specific tokens.

https://github.gds/gds/alphagov-deployment/pull/145

@robyoung robyoung merged commit 48d1fbc into master Apr 10, 2013

1 check passed

Details default Build #153 succeeded on Jenkins
@jabley jabley deleted the write-authorization branch Apr 10, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment