Skip to content

Conversation

@Partyschaum
Copy link
Contributor

  • Modified the response settings accordingly to the actual express version
  • The "Access-Control-Allow-Origin" headers are easily configurable now.

The "Access-Control-Allow-Origin" headers are easily configurable now.
@AidasK
Copy link
Member

AidasK commented Oct 1, 2014

What about these?
https://github.com/Partyschaum/flow.js/blob/fix-node-example-app/samples/Node.js/app.js#L24
https://github.com/Partyschaum/flow.js/blob/fix-node-example-app/samples/Node.js/app.js#L42

This is just an example, to make it simple, cross origin request support is commented out.

@Partyschaum
Copy link
Contributor Author

Sorry, seems I was to early with my PR. 😓

This is just an example, to make it simple, cross origin request support is commented out.

I know, but I also modified the express related response settings since the (example) app threw some deprecation warnings.

AidasK added a commit that referenced this pull request Dec 14, 2014
fix(sample): updated flow.js node backend
@AidasK AidasK merged commit 81ef017 into flowjs:master Dec 14, 2014
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