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

NEO API change #2306 #2308

Merged
merged 1 commit into from
Nov 23, 2023
Merged

NEO API change #2306 #2308

merged 1 commit into from
Nov 23, 2023

Conversation

Nargacaura
Copy link
Member

@Nargacaura Nargacaura commented Nov 23, 2023

Comme l'API de la NASA était KO depuis hier matin (vers 2h-3h environ) et l'est encore (soit ça ne renvoyait rien — aucun caractère, aucune erreur, juste "", soit ça disait qu'on était en rate limit dû à un trop grand nombre de requêtes par heure/jour — même sur une nouvelle clé crée le jour même et avec une dizaine d'essais seulement), il avait fallu que je change l'URL de l'API pour refaire fonctionner le mini-event space.

Par contre, j'ignore si:

  • la panne de l'API de la NASA n'est que temporaire (en gros, ne pas merge cette PR si elle refonctionne entre temps)
  • la NASA a désactivé cette URL pour nous forcer à utiliser l'API chez NeoWs (qui est utilisé par cette API de la NASA, évidemment)
  • la clé API est nécessaire avec NeoWs, sachant qu'elle l'était avec l'API de la NASA (elle a été gardée just in case)

@Nargacaura Nargacaura added the bug Something isn't working label Nov 23, 2023
@BastLast BastLast added this to the 4.5.0 milestone Nov 23, 2023
@BastLast
Copy link
Member

c'est fixé ?

@Nargacaura
Copy link
Member Author

Nargacaura commented Nov 23, 2023

c'est fixé ?

Yep. Ça a été testé à plusieurs reprises avec le bot alpha, avec succès. Avec le changement d'URL d'api.nasa.gov à neowsapp.com, ça a corrigé le bug. D'ailleurs, l'URL neowsapp.com est présent dans les résultats de l'API de la NASA.

@BastLast
Copy link
Member

good :)

@BastLast BastLast merged commit bedd32f into develop Nov 23, 2023
5 checks passed
@BastLast BastLast deleted the fix-space-nasa-fail branch November 23, 2023 18:03
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG]: Le mini-event space n'arrive plus à récupérer les données de l'API de la NASA
2 participants