search

bocoup / nest-weekly-review

An application for managing billing data for consulting projects
http://weekly-review.bocoup.com/
1 stars 2 forks source link

Remove authorization logic #39

Closed jugglinmike closed 9 years ago

jugglinmike commented 9 years ago

This is the first step towards gh-33 . Thanks to @bobholt

Commit message:

This client authorization has been ported to the API server, so the application code can be simplified to rely on that service.

Update the stubbing mechanism to continue to support offline development and testing.

Fully parameterize authentication providers to simplify the process of adding/removing providers in the future.

Update the fixture data to use a new access token (because the access token is now specified via a query string parameter and not a 'Set-cookie' header, whitespace should not be used).

tkellen commented 9 years ago

Woot! @jmeas and I are landing a PR soon that removes the node server entirely for Admiralty in production, serving the app from nginx. I'll link when it lands.

jugglinmike commented 9 years ago

@tkellen That's why I said "first step"; I don't plan on closing gh-33 yet

tkellen commented 9 years ago

Roger, might be worth your time to have a look at https://github.com/bocoup/admiralty/tree/deployment