Skip to content

Conversation

@ddattee
Copy link
Owner

@ddattee ddattee commented Jun 15, 2018

try out PR to check discuss if and how we can use the X-Api-Resource-State header from the API

@ronan-gloo
Copy link

ronan-gloo commented Jun 15, 2018

Adding resource stat header handler -> j'ai pas compris le but de la PR, même en regardant le code :(

  • Tu branches un middleware retry -> pourquoi la requète devrait être rejouée en fonction du header-state ?

  • Quel est le but de ce middleware, aka : quelle fonctionnalité apporte-t-il au SDK ?

@ddattee ddattee force-pushed the resource-state-header branch from ca6176f to 5f8bd85 Compare June 15, 2018 12:44
@ddattee
Copy link
Owner Author

ddattee commented Jun 15, 2018

  • retry : je ne connais pas bien les middlewares et c'est un exemple vite fait. j'ai remplace par un middleware log
  • il ajoute dans les logs l'état des ressources utilisées remonté dans l'API, pourrait servir si on ajoute un mode developpement au SDK permettant de remonter plus d'info dans les logs par exemple

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants