Project

General

Profile

Développement #47851

ovh: simplifier l'instanciation du connecteur avec la nouvelle API

Added by Valentin Deniaud about 4 years ago. Updated about 4 years ago.

Status:
Fermé
Priority:
Normal
Target version:
-
Start date:
19 October 2020
Due date:
% Done:

0%

Estimated time:
Patch proposed:
Yes
Planning:
No

Description

La configuration du connecteur est un peu plus relou avec la nouvelle API qu'avec l'ancienne :
  • Aller sur une page OVH perdue (https://eu.api.ovh.com/createToken/)
  • Rentrer id/mdp du compte OVH
  • Rentrer des infos peu utiles (titre, description)
  • Rentrer des infos critiques et toujours les mêmes (quels endpoint autoriser)
  • Obtenir 3 tokens aléatoires à copier coller, youpi
  • Si erreur quelque part, galère à débugger et très frustrant de tout refaire

Là où avant il suffisait de copier coller deux infos depuis le compte OVH.

En fait il y a un autre procédé plus simple et plus adapté, décrit ici https://docs.ovh.com/gb/en/customer/first-steps-with-ovh-api/#creation-of-your-application-keys, ça élimine 90% des infos à taper, juste copier/coller de deux tokens toujours les mêmes, login/mdp du compte OVH et ça roule.


Files

Associated revisions

Revision 4fe1eb5c (diff)
Added by Valentin Deniaud about 4 years ago

ovh: ease setup with new API (#47851)

History

#1

Updated by Valentin Deniaud about 4 years ago

#2

Updated by Benjamin Dauvergne about 4 years ago

Je n'utiliserai pas SITE_BASE_URL, plutôt self.request.build_absolute_uri(...), je sais que par convention sur nos déploiements SITE_BASE_URL est toujours disponible, mais jusqu'à présent on ne s'en est servi que quand c'était impossible de faire autrement (dans les cron/commandes en ligne).

#3

Updated by Benjamin Dauvergne about 4 years ago

  • Status changed from Solution proposée to Solution validée
#4

Updated by Valentin Deniaud about 4 years ago

Benjamin Dauvergne a écrit :

Je n'utiliserai pas SITE_BASE_URL, plutôt self.request.build_absolute_uri(...)

Yep, j'avais zappé ça à force.

#5

Updated by Valentin Deniaud about 4 years ago

  • Status changed from Solution validée to Résolu (à déployer)
commit fd87bea08e69c7523d9af5dd1ffa2250e860e3bf
Author: Valentin Deniaud <vdeniaud@entrouvert.com>
Date:   Thu Oct 15 18:18:34 2020 +0200

    ovh: ease setup with new API (#47851)
#6

Updated by Frédéric Péters about 4 years ago

  • Status changed from Résolu (à déployer) to Solution déployée

Also available in: Atom PDF