Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

allow Authorization header #2285

Merged
merged 1 commit into from Jun 16, 2017

Conversation

Projects
None yet
3 participants
Contributor

krombel commented Jun 16, 2017

The implementation of the Authorization header got implemented in #1098 but the header was not allowed (CORS)

allow Authorization header which handling got implemented in #1098
Signed-off-by: Matthias Kesler <krombel@krombel.de>

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Can one of the admins verify this patch?

Owner

erikjohnston commented Jun 16, 2017

@matrixbot ok to test

Owner

erikjohnston commented Jun 16, 2017

Thanks for this!

@erikjohnston erikjohnston merged commit 0a9315b into matrix-org:develop Jun 16, 2017

4 checks passed

Sytest Dendron (Merged PR) Build finished.
Details
Sytest Postgres (Merged PR) Build finished.
Details
Sytest SQLite (Merged PR) Build finished.
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

@krombel krombel deleted the krombel:allow_authorization_header branch Jun 16, 2017

@krombel krombel referenced this pull request in matrix-org/matrix-js-sdk Jul 4, 2017

Merged

Use access-token in header #478

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