From c93210da0fd0b0aebd2ec037d2b139fbfa0c293e Mon Sep 17 00:00:00 2001
From: Gar <gar+gh@danger.computer>
Date: Wed, 21 Sep 2022 10:53:30 -0700
Subject: [PATCH] feat: default auth-type to web

BREAKING CHANGE: the default `auth-type` config value is now `web`
---
 docs/content/using-npm/config.md                             | 2 +-
 lib/utils/config/definitions.js                              | 2 +-
 tap-snapshots/test/lib/commands/config.js.test.cjs           | 4 ++--
 tap-snapshots/test/lib/utils/config/definitions.js.test.cjs  | 2 +-
 tap-snapshots/test/lib/utils/config/describe-all.js.test.cjs | 2 +-
 5 files changed, 6 insertions(+), 6 deletions(-)

diff --git a/docs/content/using-npm/config.md b/docs/content/using-npm/config.md
index e6a7231861ba0..aa2515070afcf 100644
--- a/docs/content/using-npm/config.md
+++ b/docs/content/using-npm/config.md
@@ -217,7 +217,7 @@ exit code.
 
 #### `auth-type`
 
-* Default: "legacy"
+* Default: "web"
 * Type: "legacy", "web", "sso", "saml", "oauth", or "webauthn"
 
 NOTE: auth-type values "sso", "saml", "oauth", and "webauthn" will be
diff --git a/lib/utils/config/definitions.js b/lib/utils/config/definitions.js
index a76484b5228f5..97d203779d269 100644
--- a/lib/utils/config/definitions.js
+++ b/lib/utils/config/definitions.js
@@ -238,7 +238,7 @@ define('audit-level', {
 })
 
 define('auth-type', {
-  default: 'legacy',
+  default: 'web',
   type: ['legacy', 'web', 'sso', 'saml', 'oauth', 'webauthn'],
   // deprecation in description rather than field, because not every value
   // is deprecated
diff --git a/tap-snapshots/test/lib/commands/config.js.test.cjs b/tap-snapshots/test/lib/commands/config.js.test.cjs
index 6ed14ae5b2546..4d04fbd8fba82 100644
--- a/tap-snapshots/test/lib/commands/config.js.test.cjs
+++ b/tap-snapshots/test/lib/commands/config.js.test.cjs
@@ -20,7 +20,7 @@ exports[`test/lib/commands/config.js TAP config list --json > output matches sna
   "also": null,
   "audit": true,
   "audit-level": null,
-  "auth-type": "legacy",
+  "auth-type": "web",
   "before": null,
   "bin-links": true,
   "browser": null,
@@ -175,7 +175,7 @@ allow-same-version = false
 also = null 
 audit = true 
 audit-level = null 
-auth-type = "legacy" 
+auth-type = "web" 
 before = null 
 bin-links = true 
 browser = null 
diff --git a/tap-snapshots/test/lib/utils/config/definitions.js.test.cjs b/tap-snapshots/test/lib/utils/config/definitions.js.test.cjs
index 6eac545b2ce2d..120875db102b7 100644
--- a/tap-snapshots/test/lib/utils/config/definitions.js.test.cjs
+++ b/tap-snapshots/test/lib/utils/config/definitions.js.test.cjs
@@ -253,7 +253,7 @@ exit code.
 exports[`test/lib/utils/config/definitions.js TAP > config description for auth-type 1`] = `
 #### \`auth-type\`
 
-* Default: "legacy"
+* Default: "web"
 * Type: "legacy", "web", "sso", "saml", "oauth", or "webauthn"
 
 NOTE: auth-type values "sso", "saml", "oauth", and "webauthn" will be
diff --git a/tap-snapshots/test/lib/utils/config/describe-all.js.test.cjs b/tap-snapshots/test/lib/utils/config/describe-all.js.test.cjs
index 46910d1282e03..f7912c2db2353 100644
--- a/tap-snapshots/test/lib/utils/config/describe-all.js.test.cjs
+++ b/tap-snapshots/test/lib/utils/config/describe-all.js.test.cjs
@@ -90,7 +90,7 @@ exit code.
 
 #### \`auth-type\`
 
-* Default: "legacy"
+* Default: "web"
 * Type: "legacy", "web", "sso", "saml", "oauth", or "webauthn"
 
 NOTE: auth-type values "sso", "saml", "oauth", and "webauthn" will be