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

Handles CORS for the API. #2373

Merged
merged 1 commit into from Feb 24, 2015

Conversation

@GerardPaligot
Copy link
Member

commented Feb 23, 2015

Q R
Correction de bugs ? non
Nouvelle Fonctionnalité ? oui
Tickets (issues) concernés ZEP-17

QA : Vérifier que les headers CORS sont bien renvoyés lors d'appels à l'API.

Note : Normalement, par rapport à la documentation, le CORS devrait être géré mais je suis dans le flou total et impossible de tester la chose. Si une personne compétente à ce sujet pourrait en faire la QA pour vérifier si ça fonctionne (ou pas). Et si ça fonctionne pas, comment je pourrais tester en local convenablement.

@@ -111,6 +111,8 @@
)

MIDDLEWARE_CLASSES = (

This comment has been minimized.

Copy link
@Situphen

Situphen Feb 23, 2015

Contributor

Généralement on ne met pas les middlewares des resuirements externes après ceux de Django (comme pour INSTALlED_APPS) ?

This comment has been minimized.

Copy link
@GerardPaligot

GerardPaligot Feb 23, 2015

Author Member

Je ne mets pas souvent des commentaires mais quand j'en mets, il faut les lire. :)

This comment has been minimized.

Copy link
@Situphen

Situphen Feb 23, 2015

Contributor

Oups :-°

@SpaceFox

This comment has been minimized.

Copy link
Member

commented Feb 23, 2015

Je n'ai pas la moindre idée de ce qu'est CORS... je crois que malgré toute ma bonne volonté je vais laisser la QA à quelqu'un d'autre.

@firm1

This comment has been minimized.

Copy link
Contributor

commented Feb 24, 2015

Ta branche est déployée ici : http://vps137741.ovh.net (je la laisse la quelques heures pour ceux qui veulent tester)

De mon coté en faisant un simple curl -I http://vps137741.ovh.net/api/membres/ j'ai les entêtes suivantes :

{
"Server": "nginx/1.6.2",
"Date": "Tue, 24 Feb 2015 09:12:01 GMT",
"Content-Type": "application/json",
"Vary": "Accept, Cookie",
"Etag": "\"d8f9437e4f88b4e6e2ad0a6d770d970bfdd5bbbc689cc3b3390759d06d4f105a\"",
"Allow": "GET, POST, HEAD, OPTIONS"
}

Donc le CORS ne semble pas fonctionnel.

@GerardPaligot

This comment has been minimized.

Copy link
Member Author

commented Feb 24, 2015

@firm1 : Comme je le mentionne sur ZdS, si je fais cette requête, CORS semble appliqué :

curl -I -H "Origin: http://google.com" http://vps137741.ovh.net/api/membres/
HTTP/1.1 200 OK
Server: nginx/1.6.2
Date: Tue, 24 Feb 2015 09:29:13 GMT
Content-Type: application/json
Connection: keep-alive
Access-Control-Expose-Headers: etag, link
Vary: Accept, Cookie
ETag: "d8f9437e4f88b4e6e2ad0a6d770d970bfdd5bbbc689cc3b3390759d06d4f105a"
Allow: GET, POST, HEAD, OPTIONS
Access-Control-Allow-Origin: *
@firm1

This comment has been minimized.

Copy link
Contributor

commented Feb 24, 2015

@firm1 : Comme je le mentionne sur ZdS, si je fais cette requête, CORS semble appliqué

Je confirme, j'avais zappé l'entête.

Donc c'est fonctionnel.

Quelqu'un d'autre veut vérifier ? Parce que du coup l'objectif me semble atteind

@GerardPaligot

This comment has been minimized.

Copy link
Member Author

commented Feb 24, 2015

@firm1

This comment has been minimized.

Copy link
Contributor

commented Feb 24, 2015

QA OK alors pour nous.

@SpaceFox

This comment has been minimized.

Copy link
Member

commented Feb 24, 2015

Alors c'est l'heure du merge :)

SpaceFox added a commit that referenced this pull request Feb 24, 2015
Merge pull request #2373 from GerardPaligot/api_cors
Handles CORS for the API.

@SpaceFox SpaceFox merged commit ddcfacd into zestedesavoir:release-v1.6 Feb 24, 2015

1 check failed

continuous-integration/travis-ci/pr The Travis CI build failed
Details

@SpaceFox SpaceFox added the C-API label Feb 24, 2015

@SpaceFox SpaceFox added this to the Version 1.6 milestone Feb 24, 2015

@GerardPaligot GerardPaligot deleted the GerardPaligot:api_cors branch Feb 24, 2015

@GerardPaligot

This comment has been minimized.

Copy link
Member Author

commented Feb 24, 2015

Je suis content de ne pas avoir fait la même erreur que le throttling en m'arrachant les cheveux pendant plusieurs jours. ^^

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
4 participants
You can’t perform that action at this time.