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

Added help text for cases where origin is not present #142

Merged
merged 1 commit into from
Jun 22, 2018

Conversation

AbhishekBiswal
Copy link
Contributor

A lot of times when using the code snippet for dynamic origins, developers miss out on cases where origin is not present. For example, while using a Rest tool like Postman or making requests from a server.

README.md Outdated
@@ -115,6 +115,20 @@ app.listen(80, function () {
})
```

If you want to enable access using CuRL or any other REST Tool / Server without origin, use `corsOptions` like this:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've never seen curl written that way before. Can the capitalization just match what is used on the tool's site? https://curl.haxx.se

Copy link
Contributor Author

@AbhishekBiswal AbhishekBiswal Feb 23, 2018

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dougwilson Done.

Copy link
Contributor Author

@AbhishekBiswal AbhishekBiswal left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Done, rewrote it as curl

@dougwilson dougwilson self-assigned this Jun 22, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants