[WIP] Improve TTL management

t-geindre commented 10 years ago

Par défaut, Guzzle se moque complètement du TTL du cache et utilise les header de la réponse pour vérifier s'il doit relancer la requête.

J'ai donc ajouté une option de config sous cache : force_request_ttl. Le principe est que si on définit cette option à true alors toutes les requêtes crées auront l'option qui permet d'écraser le TTL calculé par Guzzle par le TTL définit dans le cache.

Par ailleurs, j'ai fait quelques modifications dans le GuzzleClientAdapter pour avoir une méthode générique qui permet de créer tout types de requêtes (on était limité aux GET/POST jusque là).

Je laisse en WIP pour l'instant, il va falloir que je fasse un tour dans les tests.

Avis demandés ! @omansour

close #6

mikaelrandy commented 10 years ago

:+1:

UgoSansH commented 10 years ago

:+1:

omansour commented 10 years ago

très bien. Pourrais tu penser à ajouter l'option que tu as crées dans le README ?

t-geindre commented 10 years ago

Les tests sont au vert mais ne couvrent pas tout
On a une option pour activer ou non les exceptions pour les erreurs HTTP (désactivée par défaut)
J'ai changé un peu la doc
J'ai traduit quelques commentaires de class/method. Hésitez pas à me relire, mon anglais n'est pas toujours en forme :smile:

t-geindre commented 10 years ago

Est-ce que je pousse les tests ? Sinon peut-on merger cette PR + tag ? @omansour

omansour commented 10 years ago

non c ok

BedrockStreaming / WsClientBundle

[WIP] Improve TTL management #4