bloomberg / blpapi-node

Bloomberg Open API module for node.js
http://www.bloomberglabs.com/api/about/
Other
246 stars 50 forks source link

Expose createIdentity, generateToken & sendAuthorizationRequest #49

Closed ericlu88 closed 9 years ago

ericlu88 commented 9 years ago

This PR modifies the binding layer code to be almost 1-to-1 mapping to the C++ SDK, specifically around the authorization requests. The authentication/authorization is now following the same 3 steps as documented in the developer guide:

This change also removes the default identity that used to stored inside the binding layer, and update the AuthorizationApplication.js example.

This closes #43, #44, #45

apaprocki commented 9 years ago

Can you do a quick check on the formatting to see if anything exceeds 80 cols? I'd like to keep this < 80 cols according to the BDE standard for easier reading in a standard shell.

ericlu88 commented 9 years ago

Re-formated lines > 80 columns.

apaprocki commented 9 years ago

lgtm