Skip to content
This repository has been archived by the owner on Apr 26, 2019. It is now read-only.

Fixed a trouble with incorrect reply #22

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Fixed a trouble with incorrect reply #22

Changes from all commits
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
a63ed3d
Fixed a trouble with incorrect reply
TngSam Jan 20, 2019
2dd57b7
Fixed typo
TngSam Jan 26, 2019
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
6 changes: 3 additions & 3 deletions index.js
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ module.exports = addCorsHeaders

function addCorsHeaders (request, reply) {
if (!request.headers.origin) {
return reply.continue()
return reply.continue
}

// depending on whether we have a boom or not,
Expand All @@ -12,7 +12,7 @@ function addCorsHeaders (request, reply) {
response.headers['access-control-allow-origin'] = request.headers.origin
response.headers['access-control-allow-credentials'] = 'true'
if (request.method !== 'options') {
return reply.continue()
return reply.continue
}

response.statusCode = 200
Expand All @@ -26,5 +26,5 @@ function addCorsHeaders (request, reply) {
response.headers['access-control-allow-methods'] = request.headers['access-control-request-method']
}

reply.continue()
return reply.continue
}