Project

General

Profile

Bug #3613

Fix migration for external authentication plugins

Added by Marcin Kuzminski [staff] over 3 years ago. Updated over 3 years ago.

Status:
Closed
Priority:
Normal
Assignee:
-
Category:
-
Target version:
Start date:
05.05.2016
Due date:
% Done:

0%

Estimated time:
Sorting:
Commit Number:
Affected Version:

Description

Seems used 255 filed lenght is not enough, since this upgrade was not published, we should change the migration to use fields of size 2048

History

#1 Updated by Marcin Kuzminski [staff] over 3 years ago

The migration number is 039, we also need to change the current database schema

#2 Updated by Marcin Lulek over 3 years ago

current fields are

    sa.Column('access_token', sa.String(255), default=u''),
    sa.Column('alt_token', sa.String(255), default=u''),
    sa.Column('token_secret', sa.String(255), default=u'')

for some providers - like facebook encrypted fields exceed 255 characters, they should be 512 size (or bigger) to make sure everything fits into db

#3 Updated by Johannes Bornhold over 3 years ago

What's the status of this? It looks to me as if this is not really related to CE/EE topics but a general change which we should do to improve our support for the social auth plugins.

#4 Updated by Marcin Kuzminski [staff] over 3 years ago

  • Status changed from New to Resolved

This has been done before official release

#5 Updated by Johannes Bornhold over 3 years ago

  • Target version set to v4.1
  • Status changed from Resolved to Closed

Also available in: Atom PDF