New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[WIP] Fix #4352 : remplace python-social-auth par social-auth-app-django #4357

Open
wants to merge 1 commit into
base: dev
from

Conversation

Projects
None yet
8 participants
@gustavi
Member

gustavi commented May 31, 2017

Q R
Type de modification évolution
Ticket(s) (issue(s)) concerné(s) #4352

QA

NOTE : issue en pause, cf python-social-auth/social-app-django#60

NOTE : c'est du WIP car je dois dans un premier temps tester que tout est OK sur la bêta ce que je ferai ce week-end

  • Suivre les instructions de mise en production
  • À tester sur la bêta
@Anto59290

This comment has been minimized.

Show comment
Hide comment
@Anto59290

Anto59290 Jun 1, 2017

Contributor

On a fait la même au boulot, les instructions me paraissent ok. C'est assez touchy comme migration...

Contributor

Anto59290 commented Jun 1, 2017

On a fait la même au boulot, les instructions me paraissent ok. C'est assez touchy comme migration...

@gustavi

This comment has been minimized.

Show comment
Hide comment
@gustavi

gustavi Jun 2, 2017

Member

Merci du retour ! J'ai des trucs à changer pour que Travis soit au vert, je ferai ça ASAP.

Member

gustavi commented Jun 2, 2017

Merci du retour ! J'ai des trucs à changer pour que Travis soit au vert, je ferai ça ASAP.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Jun 2, 2017

Coverage Status

Coverage decreased (-33.6%) to 55.458% when pulling 6f30616 on gustavi:fix-4352 into a2c818e on zestedesavoir:dev.

coveralls commented Jun 2, 2017

Coverage Status

Coverage decreased (-33.6%) to 55.458% when pulling 6f30616 on gustavi:fix-4352 into a2c818e on zestedesavoir:dev.

@gustavi

This comment has been minimized.

Show comment
Hide comment
@gustavi

gustavi Jun 2, 2017

Member

Issue en pause le temps que python-social-auth/social-app-django#60 soit corrigé. Je vais essayer de leur envoyer un patch si j'ai le temps.

Member

gustavi commented Jun 2, 2017

Issue en pause le temps que python-social-auth/social-app-django#60 soit corrigé. Je vais essayer de leur envoyer un patch si j'ai le temps.

@SpaceFox

This comment has been minimized.

Show comment
Hide comment
@SpaceFox

SpaceFox Sep 22, 2017

Member

Je remonte pour dire qu'ici on est grave dans la merde, parce que :

  1. On dépend toujours d'une dépendance HS.
  2. Que la dépendance a un bug (celui de Gustavi) non résolu
  3. Et que ça part mal pour la résoudre un jour parce que cette nouvelle dépendance de remplacement n'est plus développée depuis fin mai.
Member

SpaceFox commented Sep 22, 2017

Je remonte pour dire qu'ici on est grave dans la merde, parce que :

  1. On dépend toujours d'une dépendance HS.
  2. Que la dépendance a un bug (celui de Gustavi) non résolu
  3. Et que ça part mal pour la résoudre un jour parce que cette nouvelle dépendance de remplacement n'est plus développée depuis fin mai.

@Situphen Situphen added the S-Zombie label Sep 30, 2017

@motet-a

This comment has been minimized.

Show comment
Hide comment
@motet-a

motet-a Nov 6, 2017

Member

Faudrait peut-être rééssayer un coup de faire fonctionner ça vu qu’on est passé à Python 3 et Django 1.10 depuis le temps.

Member

motet-a commented Nov 6, 2017

Faudrait peut-être rééssayer un coup de faire fonctionner ça vu qu’on est passé à Python 3 et Django 1.10 depuis le temps.

@SpaceFox

This comment has been minimized.

Show comment
Hide comment
@SpaceFox

SpaceFox Nov 6, 2017

Member

Mon humble avis sur la questions : on profite de cette blague pour faire sauter les connexions via des services tiers, en affichant un mot sur la page de login que si des gens utilisaient ces services, il suffit de demander la création d'un nouveau mot de passe (normalement c'est le cas).

Member

SpaceFox commented Nov 6, 2017

Mon humble avis sur la questions : on profite de cette blague pour faire sauter les connexions via des services tiers, en affichant un mot sur la page de login que si des gens utilisaient ces services, il suffit de demander la création d'un nouveau mot de passe (normalement c'est le cas).

@motet-a

This comment has been minimized.

Show comment
Hide comment
@motet-a

motet-a Nov 6, 2017

Member

C’est vrai que c’est pas forcément la peine d’essayer de maintenir des trucs qu’on n’arrive pas à maintenir (si c’est bien ça que tu proposes).

Member

motet-a commented Nov 6, 2017

C’est vrai que c’est pas forcément la peine d’essayer de maintenir des trucs qu’on n’arrive pas à maintenir (si c’est bien ça que tu proposes).

@SpaceFox

This comment has been minimized.

Show comment
Hide comment
@SpaceFox

SpaceFox Nov 6, 2017

Member
Member

SpaceFox commented Nov 6, 2017

@gcodeur

This comment has been minimized.

Show comment
Hide comment
@gcodeur

gcodeur Nov 7, 2017

Member

Attention tout de même, cette fonction est utilisée par pas mal de membres car assez pratique. Mais si personne ne voit comment maintenir ça et résoudre nos problèmes, c'est effectivement la solution qui semble la plus indiquée.

À noter d'ailleurs qu'actuellement, le système est très incomplet : il n'y a pas d'option dans les paramètres permettant de gérer les réseaux auxquels on est connecté par exemple.

il suffit de demander la création d'un nouveau mot de passe (normalement c'est le cas).

Je confirme, ça doit être possible.

Member

gcodeur commented Nov 7, 2017

Attention tout de même, cette fonction est utilisée par pas mal de membres car assez pratique. Mais si personne ne voit comment maintenir ça et résoudre nos problèmes, c'est effectivement la solution qui semble la plus indiquée.

À noter d'ailleurs qu'actuellement, le système est très incomplet : il n'y a pas d'option dans les paramètres permettant de gérer les réseaux auxquels on est connecté par exemple.

il suffit de demander la création d'un nouveau mot de passe (normalement c'est le cas).

Je confirme, ça doit être possible.

@SpaceFox

This comment has been minimized.

Show comment
Hide comment
@SpaceFox

SpaceFox Nov 7, 2017

Member

Attention tout de même, cette fonction est utilisée par pas mal de membres car assez pratique.

On a une stat au fait sur ce point ?

Member

SpaceFox commented Nov 7, 2017

Attention tout de même, cette fonction est utilisée par pas mal de membres car assez pratique.

On a une stat au fait sur ce point ?

@gcodeur

This comment has been minimized.

Show comment
Hide comment
@gcodeur

gcodeur Nov 7, 2017

Member

L'admin Django retourne une 500, sans doute car on est toujours sur python-social-auth, mais si tu regardes la page des utilisateurs, il s'agit des membres ayant un nom renseigné (car celui-ci est ajouté à partir du service tiers), ce qui en représente quand même un certain nombre.

Member

gcodeur commented Nov 7, 2017

L'admin Django retourne une 500, sans doute car on est toujours sur python-social-auth, mais si tu regardes la page des utilisateurs, il s'agit des membres ayant un nom renseigné (car celui-ci est ajouté à partir du service tiers), ce qui en représente quand même un certain nombre.

@SpaceFox

This comment has been minimized.

Show comment
Hide comment
@SpaceFox

SpaceFox Nov 7, 2017

Member

Si ta technique de mesure est exacte, alors on a 2452 comptes avec un nom ou un prénom sur 9106, dont la quasi-totalité validés. C'est effectivement beaucoup.

Member

SpaceFox commented Nov 7, 2017

Si ta technique de mesure est exacte, alors on a 2452 comptes avec un nom ou un prénom sur 9106, dont la quasi-totalité validés. C'est effectivement beaucoup.

@Situphen Situphen removed the S-Évolution label Nov 11, 2017

@firm1

This comment has been minimized.

Show comment
Hide comment
@firm1

firm1 Aug 21, 2018

Contributor

Issue en pause le temps que python-social-auth/social-app-django#60 soit corrigé. Je vais essayer de leur envoyer un patch si j'ai le temps.

Maintenant que le bug en question est corrigé chez eux, est-ce qu'on peut envisager la migration ?

Contributor

firm1 commented Aug 21, 2018

Issue en pause le temps que python-social-auth/social-app-django#60 soit corrigé. Je vais essayer de leur envoyer un patch si j'ai le temps.

Maintenant que le bug en question est corrigé chez eux, est-ce qu'on peut envisager la migration ?

@gustavi

This comment has been minimized.

Show comment
Hide comment
@gustavi

gustavi Aug 21, 2018

Member

Je me note de regarder ça dans les jours/semaines qui viennent.

Member

gustavi commented Aug 21, 2018

Je me note de regarder ça dans les jours/semaines qui viennent.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment