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

[TECH] Tracer le message d'erreur du CDN #166

Merged
merged 1 commit into from
Nov 21, 2022

Conversation

octo-topi
Copy link
Contributor

@octo-topi octo-topi commented Nov 18, 2022

🦄 Problème

Si l'invalidation du cache échoue, on ne n'a pas le détail de la réponse

La réponse du CDN

{
  "type" : "https://www.jhipster.tech/problem/problem-with-message",
  "title" : "Bad Request",
  "status" : 400,
  "detail" : "JSON parse error: Unexpected character (',' (code 44)): expected a value; nested exception is com.fasterxml.jackson.core.JsonParseException: Unexpected character (',' (code 44)): expected a value\n at [Source: (PushbackInputStream); line: 1, column: 12]",
  "path" : "/api/cache/invalidations",
  "message" : "error.http.400"
}

Ce qui est loggé

Request failed with status code 400

🤖 Solution

Ajouter le message de réponse du CDN dans l'erreur

🌈 Remarques

Pour obtenir le format de réponse de Baleen, j'ai invalidé le cache depuis l'UI et renvoyé un payload invalide

💯 Pour tester

Vérifier que la CI passe

@octo-topi octo-topi added the team-captains This is your captain speaking label Nov 18, 2022
Copy link
Contributor

@francois2metz francois2metz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🥬 🍁 🍀 🍂 LGTM 🍂 🍀 🍁 🥬

@github-actions github-actions bot merged commit b5bf534 into main Nov 21, 2022
@github-actions github-actions bot deleted the tech-log-cdn-error-message branch November 21, 2022 08:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants