Skip to content
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

Remplacer caffeinehit/django-oauth2-provider #2229

Closed
SpaceFox opened this issue Feb 7, 2015 · 2 comments
Closed

Remplacer caffeinehit/django-oauth2-provider #2229

SpaceFox opened this issue Feb 7, 2015 · 2 comments
Labels
C-Back Concerne le back-end Django S-BUG Corrige un problème
Milestone

Comments

@SpaceFox
Copy link
Contributor

SpaceFox commented Feb 7, 2015

Comme indiqué partout dans les docs, notre fournisseur d'authentification oauth2 sur l'API est caffeinehit/django-oauth2-provider.

Ce projet n'est pas compatible avec Django 1.7. Pire, son dernier commit date du 18 mars 2014 et le domaine principal est mort, donc la PR de compatibilité Django 1.7 ne sera jamais mergée.

On a donc un problème bloquant pour le merge de la PR de passage à Django 1.7, et heureusement 3 solutions parce que cette bibliothèque est sous licence MIT.

  1. On récupère l'un des 172 forks dont au moins 5 ont déjà une branche "compatibilité Django 1.7". Si possible celui qui a l'air le plus maintenu et intégré aux outils genre Pip.
  2. On fait notre 173ème fork et 6ème branche de compatibilité Django 1.7 - sans espoir que ce soit jamais remonté dans l'upstream puisqu'il est mort.
  3. On trouve une autre bibliothèque de gestion d'oauth2.

Qu'en pensez-vous ?

@SpaceFox SpaceFox added S-BUG Corrige un problème C-Back Concerne le back-end Django Bloquant Ticket qui doit être traité avant la prochaine mise à jour labels Feb 7, 2015
@SpaceFox SpaceFox added this to the "Futur proche" (v1.x) milestone Feb 7, 2015
@firm1 firm1 removed the Bloquant Ticket qui doit être traité avant la prochaine mise à jour label Feb 8, 2015
@SpaceFox SpaceFox modified the milestones: Version 1.6, "Futur proche" (v1.x) Feb 8, 2015
@SpaceFox SpaceFox removed the Bloquant Ticket qui doit être traité avant la prochaine mise à jour label Feb 8, 2015
@firm1
Copy link
Contributor

firm1 commented Feb 9, 2015

Je ne sais pas s'il faut réouvrir l'issue ou la rennomer, mais visiblement la nouvelle bibliothèque n'est pas non plus compatible django 1.7 meme si elle a déjà plus de chance de passer dessus, ce n'est pas encore le cas.

Il y'a bien Zopieux qui a tenté une PR en ce sens, mais ça n'a pas aboutit.

@SpaceFox
Copy link
Contributor Author

SpaceFox commented Feb 9, 2015

Je lis :

Django 1.4, 1.5, 1.6, 1.7

D'autre part, la branche avec Django 1.7 semble fonctionner.

La PR de Zopieux concerne des cas particulier : est-ce qu'on peut s'assurer que ça nous concerne ?

Situphen pushed a commit to Situphen/zds-site that referenced this issue Feb 13, 2015
…olkit.

We changes the library to be compatible Django 1.7 and Python 3.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Back Concerne le back-end Django S-BUG Corrige un problème
Projects
None yet
Development

No branches or pull requests

2 participants