Skip to content

add firectl plugin - #650

Open
sambartik wants to merge 1 commit into
1Password:mainfrom
sambartik:firectl
Open

add firectl plugin#650
sambartik wants to merge 1 commit into
1Password:mainfrom
sambartik:firectl

Conversation

@sambartik

Copy link
Copy Markdown

Overview

Adds a new plugin for firectl

Type of change

  • Created a new plugin
  • Improved an existing plugin
  • Fixed a bug in an existing plugin
  • Improved contributor utilities or experience

Related Issue(s)

  • Resolves: N/A
  • Relates: N/A

How To Test

  1. Install the firectl as per https://docs.fireworks.ai/tools-sdks/firectl/firectl
  2. Make sure you have a 1Password "API Credentials" item with a field named "API Key" and with value of a fireworks.ai API key (starts with fw_ prefix).
  3. op plugin init firectl
  4. Configure it and select the 1Password item created in step 1 as a credential
  5. Run firectl users list to test if the api key is correctly passed to the binary

Note, firectl whoami command is not supported when using an API Key to authenticate, it requires a prior OAuth2 flow via firectl signin which is deliberately not supported by this plugin as it opens up a browser window with fireworks.auth.us-west-2.amazoncognito.com domain, looking very generic and untrustworthy and it also increases friction with 1Password because this domain is not typically associated with fireworks.ai item in 1Password vault.

Changelog

Added the firectl plugin to authenticate the Fireworks AI CLI with 1Password Shell Plugins.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant