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

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

Closed
Nargacaura opened this issue Nov 23, 2023 · 4 comments · Fixed by #2308
Closed
Labels
bug Something isn't working
Milestone

Comments

@Nargacaura
Copy link
Member

Description du bug.

Visiblement, la NASA a effectué des modifications sur leur API (ou elle est juste en panne?), ce qui fait qu'on arrive plus à récupérer le JSON des events spatiaux (éclipse, cycle lunaire...), donnant cette erreur:

[2023/11/22 22:27:49] SyntaxError: Unexpected end of JSON input
    at JSON.parse (<anonymous>)
    at IncomingMessage.<anonymous> (/draftbot/src/core/utils/SpaceUtils.ts:111:32)
    at IncomingMessage.emit (node:events:526:35)
    at endReadableNT (node:internal/streams/readable:1408:12)
    at processTicksAndRejections (node:internal/process/task_queues:82:21)

Du coup, les oracles (dans l'exemple ci-dessous, on a dans les 2 cas Claude, mais ça s'applique aussi aux autres) ne peuvent plus se décoller de leur télescope.

Reproduction du bug.

Exécuter le mini-event space. Tomber sur une info venant de la NASA.

Captures d'écran

Capture d’écran 2023-11-23 à 05 49 28

Sur quel shard avez vous rencontré le bug ?

Tous les shards.

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

Triste :(

@BastLast
Copy link
Member

@Nargacaura tu as moyen de check un peu plus en détail ou pas ?

@Nargacaura
Copy link
Member Author

Nargacaura commented Nov 23, 2023

Je verrai ça ce soir (en loggant la réponse donnée par la NASA avant la tentative de parsing), mais je n'avais pour l'instant que ceci comme indication que ça a foiré. Si ça fonctionne avec une autre clé API (que je testerai sur mon instance), c'est que la clé a dû être en overrate, aka elle a dépassé sa limite de requête par heure ou par jour.

@Nargacaura
Copy link
Member Author

Nargacaura commented Nov 23, 2023

Même avec une nouvelle clé, les résultats de l'API de la NASA étaient... null. Rien du tout. Quand on met l'URL sur Firefox, ça n'arrive pas à charger la page, et sur Chrome, il dit que ça redirige trop de fois.
Cependant, passer sur NeoWs fonctionne correctement alors que ça utilise aussi les données de la NASA, sans même avoir besoin de clé API, paradoxalement.

D'ailleurs, la NASA explicite sur leur site le fait que c'est un tiers, d'ailleurs, qui maintient cette API:

NeoWs (Near Earth Object Web Service) is a RESTful web service for near earth Asteroid information. With NeoWs a user can: search for Asteroids based on their closest approach date to Earth, lookup a specific Asteroid with its NASA JPL small body id, as well as browse the overall data-set.

Data-set: All the data is from the NASA JPL Asteroid team (http://neo.jpl.nasa.gov/).

This API is maintained by SpaceRocks Team: David Greenfield, Arezu Sarvestani, Jason English and Peter Baunach.

Donc peut-être changer le domaine de l'API pour implémenter NeoWs au lieu de l'API NASA.

Nargacaura added a commit that referenced this issue Nov 23, 2023
@Nargacaura Nargacaura linked a pull request Nov 23, 2023 that will close this issue
BastLast pushed a commit that referenced this issue Nov 23, 2023
@BastLast BastLast added this to the 4.5.0 milestone Nov 23, 2023
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 a pull request may close this issue.

2 participants