Prettier, bump deps

Author: fiznool

.github/workflows/nodejs.yml

@@ -2,26 +2,25 @@ name: Node.js CI
 
 on:
   push:
-    branches: [ master ]
+    branches: [master]
   pull_request:
-    branches: [ master ]
+    branches: [master]
 
 jobs:
   build:
-
     runs-on: ubuntu-latest
 
     strategy:
       matrix:
         node-version: [10.x, 12.x, 14.x]
 
     steps:
-    - uses: actions/checkout@v2
-    - name: Use Node.js ${{ matrix.node-version }}
-      uses: actions/setup-node@v1
-      with:
-        node-version: ${{ matrix.node-version }}
-    - run: npm ci
-    - run: npm test
-      env:
-        CI: true
+      - uses: actions/checkout@v2
+      - name: Use Node.js ${{ matrix.node-version }}
+        uses: actions/setup-node@v1
+        with:
+          node-version: ${{ matrix.node-version }}
+      - run: npm ci
+      - run: npm test
+        env:
+          CI: true

.prettierrc

@@ -0,0 +1,5 @@
+{
+  "bracketSpacing": true,
+  "singleQuote": true,
+  "trailingComma": "all"
+}

CHANGELOG.md

@@ -1,25 +1,35 @@
 # Change Log
+
 All notable changes to this project will be documented in this file.
 This project adheres to [Semantic Versioning](http://semver.org/).
 
 ## [2.0.1] - 2020-12-02
+
 ### Updated
+
 - Update dependencies and test against node 14.
 
 ## [2.0.0] - 2020-03-25
+
 ### Breaking
+
 - Drop support for node < 10.
 
 ## [1.1.0] - 2018-06-06
+
 ### Added
+
 - Support using a strategy which overrides the `getOAuthAccessToken` function, for example the Reddit or Spotify strategy. #10
 
 ## [1.0.0] - 2015-12-17
+
 ### Added
+
 - Allow extra params to be sent when requesting access token.
 - Use embedded `_oauth2` constructor to create new OAuth2 instance, to support instances where the `_oauth2` object is using a custom implementation.
 
 ### Removed
+
 - Dropped peerDependency on `oauth2` library, in favour of using the `_oauth2` object exposed by passport.
 - Dropped support for node.js 0.6 and 0.8, lowest supported version is now 0.10. _If you still need support for 0.6 or 0.8, please continue to use v0.4.0 of this module._
 
@@ -28,35 +38,51 @@ This project adheres to [Semantic Versioning](http://semver.org/).
 The move from 0.4 to 1.0 is non-breaking, _unless_ you are using a version of node.js lower than 0.10. In this case, you should stick to using 0.4. Otherwise, you can safely upgrade with no code changes required.
 
 ## [0.4.0] - 2015-04-01
+
 ### Added
+
 - Allow strategy to be added with an explicit name: `refresh.use(name, strategy)`.
 
 ## [0.3.1] - 2015-03-06
+
 ### Changed
+
 - Removed peer dependency on passport-oauth2, to fix npm 3 warning.
 
 ## [0.3.0] - 2015-01-27
+
 ### Added
+
 - Support strategies which use separate URLs for generating and refreshing tokens (e.g. `passport-echosign`).
 
 ## [0.2.1] - 2014-11-16
+
 ### Fixed
+
 - Fixed passport-oauth2 peer dependency link.
 
 ## [0.2.0] - 2014-11-16
+
 ### Changed
+
 - Added passport-oauth2 as a peer dependency.
 
 ## [0.1.2] - 2014-11-16
+
 ### Changed
+
 - Fixed git url.
 
 ## [0.1.1] - 2014-11-16
+
 ### Changed
+
 - Fixed README typo.
 
 ## 0.1.0 - 2014-11-16
+
 ### Added
+
 - Initial release.
 
 [2.0.1]: https://github.com/fiznool/passport-oauth2-refresh/compare/v2.0.0...v2.0.1

README.md

@@ -24,7 +24,7 @@ When setting up your passport strategies, add a call to `refresh.use()` after `p
 
 An example, using the Facebook strategy:
 
-``` js
+```js
 const passport = require('passport');
 const refresh = require('passport-oauth2-refresh');
 const FacebookStrategy = require('passport-facebook').Strategy;
@@ -48,24 +48,26 @@ refresh.use(strategy);
 
 When you need to refresh the access token, call `requestNewAccessToken()`:
 
-``` js
+```js
 const refresh = require('passport-oauth2-refresh');
-refresh.requestNewAccessToken('facebook', 'some_refresh_token', function(err, accessToken, refreshToken) {
-  // You have a new access token, store it in the user object,
-  // or use it to make a new request.
-  // `refreshToken` may or may not exist, depending on the strategy you are using.
-  // You probably don't need it anyway, as according to the OAuth 2.0 spec,
-  // it should be the same as the initial refresh token.
-
-});
-
+refresh.requestNewAccessToken(
+  'facebook',
+  'some_refresh_token',
+  function (err, accessToken, refreshToken) {
+    // You have a new access token, store it in the user object,
+    // or use it to make a new request.
+    // `refreshToken` may or may not exist, depending on the strategy you are using.
+    // You probably don't need it anyway, as according to the OAuth 2.0 spec,
+    // it should be the same as the initial refresh token.
+  },
+);
 ```
 
 ### Specific name
 
 Instead of using the default `strategy.name`, you can setup `passport-oauth2-refresh` to use an specific name instead.
 
-``` js
+```js
 // Setup
 passport.use('gmail', googleStrategy);
 
@@ -79,7 +81,7 @@ This can be useful if you'd like to reuse strategy objects but under a different
 
 Some endpoints require additional parameters to be sent when requesting a new access token. To send these parameters, specify the parameters when calling `requestNewAccessToken` as follows:
 
-``` js
+```js
 const extraParams = { some: 'extra_param' };
 refresh.requestNewAccessToken('gmail', 'some_refresh_token', extraParams, done);
 ```

lib/refresh.js

@@ -16,22 +16,24 @@ AuthTokenRefresh._strategies = {};
  * @param {String|Strategy} name
  * @param {Strategy} passport strategy
  */
-AuthTokenRefresh.use = function(name, strategy) {
-  if(arguments.length === 1) {
+AuthTokenRefresh.use = function (name, strategy) {
+  if (arguments.length === 1) {
     // Infer name from strategy
     strategy = name;
     name = strategy && strategy.name;
   }
 
-  if(strategy == null) {
+  if (strategy == null) {
     throw new Error('Cannot register: strategy is null');
   }
 
-  if(!name) {
-    throw new Error('Cannot register: name must be specified, or strategy must include name');
+  if (!name) {
+    throw new Error(
+      'Cannot register: name must be specified, or strategy must include name',
+    );
   }
 
-  if(!strategy._oauth2) {
+  if (!strategy._oauth2) {
     throw new Error('Cannot register: not an OAuth2 strategy');
   }
 
@@ -50,20 +52,22 @@ AuthTokenRefresh.use = function(name, strategy) {
       strategy._oauth2._baseSite,
       strategy._oauth2._authorizeUrl,
       strategy._refreshURL || strategy._oauth2._accessTokenUrl,
-      strategy._oauth2._customHeaders)
+      strategy._oauth2._customHeaders,
+    ),
   };
 
   // Some strategies overwrite the getOAuthAccessToken function to set headers
   // https://github.com/fiznool/passport-oauth2-refresh/issues/10
-  AuthTokenRefresh._strategies[name].refreshOAuth2.getOAuthAccessToken = strategy._oauth2.getOAuthAccessToken;
+  AuthTokenRefresh._strategies[name].refreshOAuth2.getOAuthAccessToken =
+    strategy._oauth2.getOAuthAccessToken;
 };
 
 /**
  * Check if a strategy is registered for refreshing.
  * @param  {String}  name Strategy name
  * @return {Boolean}
  */
-AuthTokenRefresh.has = function(name) {
+AuthTokenRefresh.has = function (name) {
   return !!AuthTokenRefresh._strategies[name];
 };
 
@@ -78,16 +82,21 @@ AuthTokenRefresh.has = function(name) {
  *                                 params to use when requesting the token.
  * @param  {Function} done         Callback when all is done.
  */
-AuthTokenRefresh.requestNewAccessToken = function(name, refreshToken, params, done) {
-  if(arguments.length === 3) {
+AuthTokenRefresh.requestNewAccessToken = function (
+  name,
+  refreshToken,
+  params,
+  done,
+) {
+  if (arguments.length === 3) {
     done = params;
     params = {};
   }
 
   // Send a request to refresh an access token, and call the passed
   // callback with the result.
   const strategy = AuthTokenRefresh._strategies[name];
-  if(!strategy) {
+  if (!strategy) {
     return done(new Error('Strategy was not registered to refresh a token'));
   }