feat: allow stdin for sfdx-url (#886)

command-snapshot.json

@@ -82,16 +82,26 @@
   {
     "command": "org:login:sfdx-url",
     "plugin": "@salesforce/plugin-auth",
-    "flags": ["alias", "json", "loglevel", "no-prompt", "set-default", "set-default-dev-hub", "sfdx-url-file"],
+    "flags": [
+      "alias",
+      "json",
+      "loglevel",
+      "no-prompt",
+      "set-default",
+      "set-default-dev-hub",
+      "sfdx-url-file",
+      "sfdx-url-stdin"
+    ],
     "alias": ["force:auth:sfdxurl:store", "auth:sfdxurl:store"],
-    "flagChars": ["a", "d", "f", "p", "s"],
+    "flagChars": ["a", "d", "f", "p", "s", "u"],
     "flagAliases": [
       "noprompt",
       "setalias",
       "setdefaultdevhub",
       "setdefaultdevhubusername",
       "setdefaultusername",
-      "sfdxurlfile"
+      "sfdxurlfile",
+      "sfdxurlstdin"
     ]
   },
   {

messages/sfdxurl.store.md

@@ -1,6 +1,6 @@
 # summary
 
-Authorize an org using a Salesforce DX authorization URL stored in a file.
+Authorize an org using a Salesforce DX authorization URL stored in a file or through standard input (stdin).
 
 # description
 
@@ -18,10 +18,16 @@ NOTE: The "<%= config.bin %> org display --verbose" command displays the refresh
 
 You can also create a JSON file that has a top-level property named sfdxAuthUrl whose value is the authorization URL. Finally, you can create a normal text file that includes just the URL and nothing else.
 
+Alternatively, you can pipe the SFDX authorization URL through standard input by using the --sfdx-url-stdin flag and providing the '-' character as the value.
+
 # flags.sfdx-url-file.summary
 
 Path to a file that contains the Salesforce DX authorization URL.
 
+# flags.sfdx-url-stdin.summary
+
+Specify '-' as this flag's value to pipe the Salesforce DX authorization URL through standard input (stdin).
+
 # examples
 
 - Authorize an org using the SFDX authorization URL in the files/authFile.json file:
@@ -31,3 +37,7 @@ Path to a file that contains the Salesforce DX authorization URL.
 - Similar to previous example, but set the org as your default and give it an alias MyDefaultOrg:
 
   <%= config.bin %> <%= command.id %> --sfdx-url-file files/authFile.json --set-default --alias MyDefaultOrg
+
+- Pipe the SFDX authorization URL from stdin by specifying the '-' value.
+
+  <%= "\n $ echo url | " + config.bin %> <%= command.id %> --sfdx-url-stdin -

src/commands/org/login/sfdx-url.ts

@@ -33,9 +33,19 @@ export default class LoginSfdxUrl extends AuthBaseCommand<AuthFields> {
     'sfdx-url-file': Flags.file({
       char: 'f',
       summary: messages.getMessage('flags.sfdx-url-file.summary'),
-      required: true,
+      required: false,
       deprecateAliases: true,
       aliases: ['sfdxurlfile'],
+      exactlyOne: ['sfdx-url-file', 'sfdx-url-stdin'],
+    }),
+    'sfdx-url-stdin': Flags.file({
+      char: 'u',
+      summary: messages.getMessage('flags.sfdx-url-stdin.summary'),
+      required: false,
+      deprecateAliases: true,
+      aliases: ['sfdxurlstdin'],
+      allowStdin: 'only',
+      exactlyOne: ['sfdx-url-file', 'sfdx-url-stdin'],
     }),
     'set-default-dev-hub': Flags.boolean({
       char: 'd',
@@ -71,15 +81,21 @@ export default class LoginSfdxUrl extends AuthBaseCommand<AuthFields> {
     if (await this.shouldExitCommand(flags['no-prompt'])) return {};
 
     const authFile = flags['sfdx-url-file'];
+    const authStdin = flags['sfdx-url-stdin'];
+    let sfdxAuthUrl: string;
 
-    const sfdxAuthUrl = authFile.endsWith('.json')
-      ? await getUrlFromJson(authFile)
-      : await fs.readFile(authFile, 'utf8');
+    if (authFile) {
+      sfdxAuthUrl = authFile.endsWith('.json') ? await getUrlFromJson(authFile) : await fs.readFile(authFile, 'utf8');
 
-    if (!sfdxAuthUrl) {
-      throw new Error(
-        `Error getting the auth URL from file ${authFile}. Please ensure it meets the description shown in the documentation for this command.`
-      );
+      if (!sfdxAuthUrl) {
+        throw new Error(
+          `Error getting the auth URL from file ${authFile}. Please ensure it meets the description shown in the documentation for this command.`
+        );
+      }
+    } else if (authStdin) {
+      sfdxAuthUrl = authStdin;
+    } else {
+      throw new Error('SFDX Auth URL not found.');
     }
 
     const oauth2Options = AuthInfo.parseSfdxAuthUrl(sfdxAuthUrl);

test/commands/org/login/login.sfdx-url.test.ts

@@ -10,7 +10,7 @@ import { AuthFields, AuthInfo, Global, Mode } from '@salesforce/core';
 import { MockTestOrgData, TestContext } from '@salesforce/core/lib/testSetup.js';
 import { expect } from 'chai';
 import { Config } from '@oclif/core';
-import { StubbedType, stubInterface } from '@salesforce/ts-sinon';
+import { StubbedType, stubInterface, stubMethod } from '@salesforce/ts-sinon';
 import { SfCommand } from '@salesforce/sf-plugins-core';
 import LoginSfdxUrl from '../../../../src/commands/org/login/sfdx-url.js';
 
@@ -213,4 +213,33 @@ describe('org:login:sfdx-url', () => {
     await store.run();
     expect(authInfoStub.save.called);
   });
+
+  it('should error out when neither file or url are provided', async () => {
+    await prepareStubs();
+    const store = new LoginSfdxUrl([], {} as Config);
+    try {
+      const response = await store.run();
+      expect.fail(`Should have thrown an error. Response: ${JSON.stringify(response)}`);
+    } catch (e) {
+      expect((e as Error).message).to.includes(
+        'Exactly one of the following must be provided: --sfdx-url-file, --sfdx-url-stdin'
+      );
+    }
+  });
+
+  it('should return auth fields when using stdin', async () => {
+    await prepareStubs();
+    const sfdxAuthUrl = 'force://PlatformCLI::[email protected]';
+    const flagOutput = {
+      flags: {
+        'no-prompt': false,
+        'sfdx-url-file': '',
+        'sfdx-url-stdin': sfdxAuthUrl,
+      },
+    };
+    const store = new LoginSfdxUrl(['-u', '-'], {} as Config);
+    stubMethod($$.SANDBOX, store, 'parse').resolves(flagOutput);
+    const response = await store.run();
+    expect(response.username).to.equal(testData.username);
+  });
 });