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

Is Gandyn affected by the recent API change? #18

Open
mossroy opened this issue Aug 2, 2017 · 0 comments
Open

Is Gandyn affected by the recent API change? #18

mossroy opened this issue Aug 2, 2017 · 0 comments

Comments

@mossroy
Copy link
Contributor

mossroy commented Aug 2, 2017

I received an email from Gandi today, explaining they had to modify their API for a security issue.
Does it affect Gandyn?

Excerpt of the email (in French) :

Il est nécessaire de vérifier votre code et plus précisément
l'utilisation ou non de la méthode
"gandi.api.domains.zone.record::list".

Le cas échéant, notez que l'utilisation de cette méthode a été modifiée,
en particulier le type du champs 'id' remonté par cette fonction. Votre
code doit dorénavant accepter le fait que le champs 'id' soit une chaîne
de charactères (string) et non plus un entier (i4 en xmlrpc).

Les deux seules fonctions utilisant cet 'id' en entrée sont :

  • gandi.api.domains.zone.record::update
  • gandi.api.domains.zone.record::list (filtre sur l'id)

Afin d'assurer la rétrocompatibilité optimale, ces deux fonctions
acceptent toujours un type "entier" en entrée pour le champs 'id' en
plus du type "chaîne de charactères". Il est cependant fortement
recommandé d'utiliser le type "chaîne de charactères".

From what I understand, I don't think Gandyn is affected, as it is written in Python and relies on dynamic typing. I suppose it accepts ids as string or integer.

Could you confirm?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant