Skip to content
This repository has been archived by the owner. It is now read-only.

feat(OpenID): Enable CORS on the /.well-known/openid-configuration endpoint. #5730

Merged
merged 1 commit into from Nov 16, 2017

Conversation

@shane-tomlinson
Copy link
Member

@shane-tomlinson shane-tomlinson commented Nov 16, 2017

Allow a route to declare a cors property that is used
to create a cors middleware.

fixes #5453

@vladikoff, @rfk - r?

I opened this against train-100 so that @vladikoff has it ready for #5675

…dpoint.

Allow a route to declare a `cors` property that is used
to create a cors middleware.

fixes #5453
@rfk
rfk approved these changes Nov 16, 2017
Copy link
Member

@rfk rfk left a comment

Looks good to me


res.json(openidConfig);
return {
cors: {

This comment has been minimized.

@vladikoff

vladikoff Nov 16, 2017
Contributor

👍

@vladikoff vladikoff merged commit 98c5b41 into train-100 Nov 16, 2017
4 checks passed
4 checks passed
ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
coverage/coveralls First build on issue-5453-cors-openid-configuration at 97.804%
Details
@vladikoff vladikoff deleted the issue-5453-cors-openid-configuration branch Nov 16, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants