Description
Proposal
Add a --client-store-temporary-credential flag to snow connection add so the whole connection, including this setting, can be created in a single command:
snow connection add --connection-name oauth --account xxxxxxx-xxxxxxx --user xxx --authenticator OAUTH_AUTHORIZATION_CODE --role xxx --client-store-temporary-credential --no-interactive
Flag name, type, default: --client-store-temporary-credential, boolean flag (is_flag=True), default None/unset (only written to config.toml when explicitly passed) — matching the naming and behavior of the existing global connection option of the same name (ClientStoreTemporaryCredentialOption in flags.py), and following the precedent of --server-session-keep-alive, which was added to connection add the same way.
Where it lives: existing snow connection add command, no new group/command needed.
Output/lifecycle: no change to output format or result type; not tied to any PrPr/PuPr/GA staging — it's a plain config-writing addition, consistent with the other boolean connection flags already supported by add.
Alternative considered
Keep editing config.toml by hand after running connection add. Rejected because it defeats the purpose of having a non-interactive, scriptable connection add command.
Status
I already have an implementation ready. If this proposal is approved, I plan to open a pull request:
yuuu@1492a51
Context
When adding an OAuth-based connection, I currently run:
snow connection add --connection-name oauth --account xxxxxxx-xxxxxxx --user xxx --authenticator OAUTH_AUTHORIZATION_CODE --role xxx --no-interactive
To make the browser-based OAuth flow skip re-authentication on every session, I then have to manually edit config.toml and add:
client_store_temporary_credential = true
This manual step breaks the fully CLI-driven workflow I'd otherwise have — everything else about setting up the connection can be scripted, but this one setting cannot.
Description
Proposal
Add a
--client-store-temporary-credentialflag tosnow connection addso the whole connection, including this setting, can be created in a single command:Flag name, type, default:
--client-store-temporary-credential, boolean flag (is_flag=True), defaultNone/unset (only written toconfig.tomlwhen explicitly passed) — matching the naming and behavior of the existing global connection option of the same name (ClientStoreTemporaryCredentialOptioninflags.py), and following the precedent of--server-session-keep-alive, which was added toconnection addthe same way.Where it lives: existing
snow connection addcommand, no new group/command needed.Output/lifecycle: no change to output format or result type; not tied to any PrPr/PuPr/GA staging — it's a plain config-writing addition, consistent with the other boolean connection flags already supported by
add.Alternative considered
Keep editing
config.tomlby hand after runningconnection add. Rejected because it defeats the purpose of having a non-interactive, scriptableconnection addcommand.Status
I already have an implementation ready. If this proposal is approved, I plan to open a pull request:
yuuu@1492a51
Context
When adding an OAuth-based connection, I currently run:
To make the browser-based OAuth flow skip re-authentication on every session, I then have to manually edit
config.tomland add:This manual step breaks the fully CLI-driven workflow I'd otherwise have — everything else about setting up the connection can be scripted, but this one setting cannot.