Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Fix for error handling when using more than one 2-legged OAuth strategy #114

Open
wants to merge 3 commits into
from

Conversation

Projects
None yet
1 participant

chiefy commented Dec 4, 2012

We have a situation where we need to listen on https and http and authenticate via OAuth-two-legged. Currently the strategy will try to write http headers each time it errors if there is more than one strategy being used causing an exception. This works around it by adding some error data to an later accessible object.

Christopher Najewicz and others added some commits Nov 30, 2012

Merge branch 'master' of git://github.com/ciaranj/connect-auth
* 'master' of git://github.com/ciaranj/connect-auth:
  allow forcing the display of the OAuth approval screen
  added proper license
  Updated readme
  Some documentation fixes
  Minor CSS tweak to make images look less awful in the example app
  Adds new Mozilla Persona Strategy
  Removed extraneous .gitmodules file that was breaking Travis CI
  Removed another require('sys')
  Bumping version to 0.6.0
  Bump version to 0.5.4
  add support for offline access
  Use oauth_verifier with getOAuthAccessToken for twitter.
  Use Connect 2.7
  [fix] Changed require('sys') to require('util') for compatibility with node v0.8
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment