[PATCH 4/6] Create the /api/ page providing the API token and information

Pierre-Yves pingou at pingoured.fr
Tue Jan 8 08:32:33 UTC 2013


On Tue, Jan 08, 2013 at 02:45:17AM -0500, Bohuslav Kabrda wrote:
> ----- Original Message -----
> > The API page aims at providing information about the API itself and
> > how to use it.
> > Additionally, if you are logged-in, this page gives you your own
> > API token (needed to call the API), its expiration date (as each
> > token is valid only 30 days) and a way to generate a new token.
> > ---
> 
> Hey Pingou,
> thanks for the patches. The first three are ok and I'll apply them.
> As for the token authentication, I'm not generally opposed to that, but:
> 1) the alembic migration fails with
> 'Cannot add a NOT NULL column with default value NULL u'ALTER TABLE user ADD COLUMN api_token'
> so the migration must contain a sane value for already existing users. Also the migration should create an expired token, I think. Just to make sure we're safe with the auto-inserted empty token...

Good point, I thought about it but forgot when I wrote the patch. Good idea with
the expired token, that's an easy solution: default token 'abc' expiration date:
'2000-01-01' and problem solve :)

> 2) What exactly do we need the token for? Anything else than user comfort? If not, I think that a cookie based permanent login would be a good standard solution for this problem.

The idea of the token is to allow creating a CLI for coprs, this way one can
have a ~/.config/coprs file containing the token and call the API from the
website to perform the different action we would like to see in the CLI.

Does this make sense ?


Regards,
Pierre


More information about the copr-devel mailing list