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

Adding a 'proxies' section to the Jefferson configuration #2

Merged
merged 9 commits into from
Mar 18, 2015

Conversation

darthtrevino
Copy link
Contributor

  • Proxies are objects with an 'init' method. The init method accepts a delegate middleware and returns a middleware function. It is expected that the proxy invokes the delegate.

@darthtrevino darthtrevino changed the title Adding a 'proxy' section to the Jefferson configuration Adding a 'proxies' section to the Jefferson configuration Mar 18, 2015
@natoverse
Copy link
Contributor

Looks good. I love mocha - makes that sort of test so easy.

@darthtrevino
Copy link
Contributor Author

Mocha is great! We're also using supertest to directly test the express app object's responses https://github.com/visionmedia/supertest

@natoverse
Copy link
Contributor

You should put a Promise example in the readme. I think proxies are a good compromise on the seam - no forced dependency, but the power to convert your middleware as needed. Means you can mix and match middleware types too, so you can pull in other libs that may not be promisified.

@darthtrevino
Copy link
Contributor Author

It may be worthwhile to capture some common proxies that we consider to be useful and store them in a 'proxies' folder. Then clients could use require('express-jefferson/promise-proxy')

@natoverse
Copy link
Contributor

👍 to boilerplate proxies

darthtrevino and others added 4 commits March 18, 2015 11:24
Adding promise conversion proxy example
* Adding a promise-handling proxy
* Combining src and test
…-proxies

* '1-proxies' of github.com:atsid/express-jefferson:
  Update README.md
  Update README.md
@coveralls
Copy link

Coverage Status

Coverage decreased (-3.23%) to 96.77% when pulling ab37dfb on 1-proxies into a19da5a on master.

…of snake_casing to match the library idioms.

* Updating the npmignore so that coverage isn't published
* Updating the output of the library code so that require('') on sub-paths will behave normally
@darthtrevino
Copy link
Contributor Author

I think this PR is about ready to roll. When travis gives me the greenlight I'll merge it.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 40a4979 on 1-proxies into a19da5a on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 40a4979 on 1-proxies into a19da5a on master.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 100.0% when pulling 40a4979 on 1-proxies into a19da5a on master.

darthtrevino added a commit that referenced this pull request Mar 18, 2015
Adding a 'proxies' section to the Jefferson configuration
@darthtrevino darthtrevino merged commit f73dcfa into master Mar 18, 2015
@darthtrevino darthtrevino deleted the 1-proxies branch March 25, 2015 17:34
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.

3 participants