On Tue, 2013-03-12 at 02:59 -0400, Bohuslav Kabrda wrote:
----- Original Message -----
> Repository :
http://git.fedorahosted.org/cgit/copr.git
>
> On branch : master
>
> >---------------------------------------------------------------
>
> commit ce9e2e623f122ffcb1dd1f81cdc8553daa8c48e5
> Author: Pierre-Yves Chibon <pingou(a)pingoured.fr>
> Date: Tue Jan 8 20:08:30 2013 +0100
>
> Move the API token length in the config file and adjust db
> accordingly
>
> Up to now the API token length was hard-coded in the code to 30
> chars
> long. With this commit this limit is now set in the configuration
> file of the application.
> In order for this to make sense, we need to bump the size of the
> field
> in the database to 255 (should be more than enough).
>
> NOTE: the alembic upgrade won't work on sqlite which consider
> pretty
> much only two types of field: numeric or text. So it has no
> problem
> storing a 100 chars long string in a varchar(40).
>
>
> >---------------------------------------------------------------
>
> coprs_frontend/coprs/views/misc.py | 1 +
> 1 files changed, 1 insertions(+), 0 deletions(-)
>
> diff --git a/coprs_frontend/coprs/views/misc.py
> b/coprs_frontend/coprs/views/misc.py
> index d7eb73e..383b832 100644
> --- a/coprs_frontend/coprs/views/misc.py
> +++ b/coprs_frontend/coprs/views/misc.py
> @@ -113,3 +113,4 @@ def backend_authenticated(f):
> return 'You have to provide the correct password', 401
> return f(*args, **kwargs)
> return decorated_function
Hmm, are you sure you added all the needed files to the commit? It doesn't seem so.
Yeah, I'm not quite sure what happened there, I think I forgot to rebase
on the latest master and it tried to push a commit that was already
there. I didn't dare the force push to remove it. Sorry, I should have
checked what I was pushing before doing so.
Pierre