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

Fix to issue #80 #88

Merged
merged 6 commits into from Oct 6, 2017

Conversation

Projects
None yet
1 participant
@mebjas
Owner

mebjas commented Sep 23, 2017

#80

Changing transport of token in case of XHR POST to request header. On server side, for each POST request the library checks for token in POST payload then in request.

mebjas added some commits Sep 17, 2017

fix for issue 80
#80
Added support for content-type = application json in both client and
server side.
transport of token moved to headers for XHR post
On js side, token is added to request header in case of POST request.
On server side - POST request, it looks for data in header, if failed
then looks for data in post payload.

@mebjas mebjas added this to the v1.0.0 milestone Sep 23, 2017

mebjas added some commits Sep 23, 2017

added support to case when apache_request_headers is missing
server now looks for token in payload else from header.
In case of header first from - apache_request_headers else from
superglobal $_SERVER
@mebjas

This comment has been minimized.

Owner

mebjas commented Oct 5, 2017

TODO: add test cases to check this case

@mebjas mebjas merged commit 88cfff1 into master Oct 6, 2017

2 checks passed

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

@mebjas mebjas deleted the dev-master-issue80 branch Oct 6, 2017

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