Feat/env variables #183
Feat/env variables #183
Conversation
vvatelot
force-pushed
the
feat/env-variables
branch
from
c3e0a3e
to
bee9ab2
Compare
| [api] | ||
| key = "3037e7e96fmsh12bedced9f019f8p1cd804jsn4967070f8bda" | ||
| baseurl = "https://ecoindex.p.rapidapi.com/v1/" | ||
| host = "ecoindex.p.rapidapi.com" |
There was a problem hiding this comment.
❓ question: Ne faudrait il pas plutôt mettre des valeurs nulles ici ? Et on documente comment lancer l'application avec les valeurs qui vont bien... ça me paraitrait plus propre. Tu en penses quoi @yaaax ?
There was a problem hiding this comment.
Pourquoi pas ! On peut en profiter pour générer un nouvelle clé ? Histoire d'éviter d'avoir une clé valide dans le code ?
Même si cette clé reste lisible pour ceux qui regardent le contenu des requêtes… (la limitation restant la vérification de l'hôte et les 5 tentatives par jour, vérifications gérées côté serveur)
Ça me va !
|
J'ai pu tester, ça marche bien ! C'est top ! |
vvatelot
force-pushed
the
feat/env-variables
branch
from
bee9ab2
to
1aa1c68
Compare
vvatelot
force-pushed
the
feat/env-variables
branch
from
1aa1c68
to
2e94430
Compare
vvatelot
force-pushed
the
feat/env-variables
branch
from
2e94430
to
62ba648
Compare
|
Cette pull request a été marquée comme inactive car elle n'a pas eu de mise à jour depuis 30 jours. Si elle n'est pas mise à jour d'ici 7 jours, elle sera fermée automatiquement. Pour éviter qu'elle soit fermée, vous pouvez ajouter le label |
Bumps [ky](https://github.com/sindresorhus/ky) from 0.33.3 to 1.2.0. - [Release notes](https://github.com/sindresorhus/ky/releases) - [Commits](sindresorhus/ky@v0.33.3...v1.2.0) --- updated-dependencies: - dependency-name: ky dependency-type: direct:production update-type: version-update:semver-major ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com> Feat/env variables (#183) * Get api settings from config (or environment variables) * Remove debugger * format code --------- Co-authored-by: Vincent Vatelot <vincent.vatelot@ik.me> Revert "Feat/env variables" (#307) * Revert "Feat/env variables (#183)" This reverts commit 90d9c02. * chore: revert env variables
Move hard coded variables to config params:
All params can be set through environment variables (see Configure with Environment Variables)
Those new variables can be set with those environment variables:
HUGO_PARAMS_api_baseURLHUGO_PARAMS_api_hostHUGO_PARAMS_api_apiKey