netlify-cli 6.14.19 → 6.14.25

package/src/utils/gh-auth.js

@@ -13,6 +13,14 @@ const { openBrowser } = require('./open-browser')
 
 const SERVER_PORT = 3000
 
+/**
+ * @typedef Token
+ * @type {object}
+ * @property {string} user - The username that is associated with the token
+ * @property {string} token - The actual token value starting with `gho_`
+ * @property {string} provider - The Provider where the token is associated with ('github').
+ */
+
 const promptForAuthMethod = async () => {
   const authChoiceNetlify = 'Authorize with GitHub through app.netlify.com'
   const authChoiceToken = 'Authorize with a GitHub personal access token'
@@ -34,14 +42,7 @@ const promptForAuthMethod = async () => {
 
 /**
  * Authenticate with the netlify app
- * @returns {Promise<Record<string,string>} Returns a Promise with an object of the following shape
- * ```
- * {
- *   user: 'spongebob,
- *   token: 'gho_some-token',
- *   provider: 'github'
- * }
- * ```
+ * @returns {Promise<Token>} Returns a Promise with a token object
  */
 const authWithNetlify = async () => {
   const port = await getPort({ port: SERVER_PORT })
@@ -95,6 +96,10 @@ const getPersonalAccessToken = async () => {
   return { token }
 }
 
+/**
+ * Authenticate with the netlify app
+ * @returns {Promise<Token>} Returns a Promise with a token object
+ */
 const authWithToken = async () => {
   const { token } = await getPersonalAccessToken()
   if (token) {
@@ -108,6 +113,10 @@ const authWithToken = async () => {
   throw error
 }
 
+/**
+ * Get a github token
+ * @returns {Promise<Token>} Returns a Promise with a token object
+ */
 const getGitHubToken = async () => {
   log('')
 

package/src/utils/init/config-github.js

@@ -2,10 +2,18 @@ const { Octokit } = require('@octokit/rest')
 const chalk = require('chalk')
 
 const { error: failAndExit, log } = require('../command-helpers')
-const ghauth = require('../gh-auth')
+const { getGitHubToken: ghauth } = require('../gh-auth')
 
 const { createDeployKey, formatErrorMessage, getBuildSettings, saveNetlifyToml, setupSite } = require('./utils')
 
+/**
+ * @typedef Token
+ * @type {object}
+ * @property {string} user - The username that is associated with the token
+ * @property {string} token - The actual token value.
+ * @property {string} provider - The Provider where the token is associated with ('github').
+ */
+
 const formatRepoAndOwner = ({ repoName, repoOwner }) => ({
   name: chalk.magenta(repoName),
   owner: chalk.magenta(repoOwner),
@@ -13,14 +21,27 @@ const formatRepoAndOwner = ({ repoName, repoOwner }) => ({
 
 const PAGE_SIZE = 100
 
-const isValidToken = (token) => token && token.user && token.token
-
+/**
+ * Get a valid github token
+ * @returns {string}
+ */
 const getGitHubToken = async ({ globalConfig }) => {
   const userId = globalConfig.get('userId')
+
+  /** @type {Token} */
   const githubToken = globalConfig.get(`users.${userId}.auth.github`)
 
-  if (isValidToken(githubToken)) {
-    return githubToken.token
+  if (githubToken && githubToken.user && githubToken.token) {
+    try {
+      const octokit = getGitHubClient(githubToken.token)
+      const { status } = await octokit.rest.users.getAuthenticated()
+      if (status < 400) {
+        return githubToken.token
+      }
+    } catch {
+      log(chalk.yellow('Token is expired or invalid!'))
+      log('Generating a new Github token...')
+    }
   }
 
   const newToken = await ghauth()
@@ -28,12 +49,15 @@ const getGitHubToken = async ({ globalConfig }) => {
   return newToken.token
 }
 
-const getGitHubClient = ({ token }) => {
-  const octokit = new Octokit({
+/**
+ * Retrieves the Github octokit client
+ * @param {string} token
+ * @returns {Octokit}
+ */
+const getGitHubClient = (token) =>
+  new Octokit({
     auth: `token ${token}`,
   })
-  return octokit
-}
 
 const addDeployKey = async ({ api, octokit, repoName, repoOwner }) => {
   log('Adding deploy key to repository...')
@@ -172,7 +196,7 @@ const addNotificationHooks = async ({ api, siteId, token }) => {
   log(`Netlify Notification Hooks configured!`)
 }
 
-module.exports = async function configGithub({ context, repoName, repoOwner, siteId }) {
+const configGithub = async ({ context, repoName, repoOwner, siteId }) => {
   const { netlify } = context
   const {
     api,
@@ -193,7 +217,7 @@ module.exports = async function configGithub({ context, repoName, repoOwner, sit
   })
   await saveNetlifyToml({ repositoryRoot, config, configPath, baseDir, buildCmd, buildDir, functionsDir })
 
-  const octokit = getGitHubClient({ token })
+  const octokit = getGitHubClient(token)
   const [deployKey, githubRepo] = await Promise.all([
     addDeployKey({ api, octokit, repoOwner, repoName }),
     getGitHubRepo({ octokit, repoOwner, repoName }),
@@ -223,3 +247,5 @@ module.exports = async function configGithub({ context, repoName, repoOwner, sit
   log()
   await addNotificationHooks({ siteId, api, token })
 }
+
+module.exports = { configGithub, getGitHubToken }

package/src/utils/init/config.js

@@ -2,7 +2,7 @@ const chalk = require('chalk')
 
 const { log } = require('../command-helpers')
 
-const configGithub = require('./config-github')
+const { configGithub } = require('./config-github')
 const configManual = require('./config-manual')
 
 const logSuccess = (repoData) => {