search

gladiaio / gladia-samples

23 stars 5 forks source link

feat: replace x_gladia_key to x-gladia-key #12

Closed amorin-gladia closed 9 months ago

amorin-gladia commented 9 months ago

Pour le coup ça fait bizarre ...

Vous en pensez quoi ?

nmorel commented 9 months ago

Pour le coup ça fait bizarre ...

Vous en pensez quoi ?

Je suis d'accord, ça fait bizarre. On se retrouve avec un mélange de snake_case et kebab-case au sein du même objet JSON.
Pour moi, le x-gladia-key est adapté pour l'api car c'est un header HTTP mais moins dans le cas d'une clé d'objet JSON.
Surtout le préfixe x- qui est très spécifique header http.

Pourquoi pas gladia_key ou api_key ?
Après, pour info, on ne devrait plus avoir ce débat avec la v2 car on passera par un call api rest classique pour s'authentifier (plus de key ou de config à passer à la connexion WS).

amorin-gladia commented 9 months ago

Je suis d'accords avec toi Nicolas. @jsoto-gladia Je close ?

amorin-gladia commented 9 months ago

not relevant anymore