Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

spike/cli improvements #239

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

spike/cli improvements #239

wants to merge 2 commits into from

Conversation

bangarang
Copy link
Collaborator

@bangarang bangarang commented Dec 16, 2024

Adds an API wrapper that can be called easily from the CLI:

Screenshot 2024-12-16 at 1 50 29 PM

Comment on lines +148 to +151
program
.command('api <resource> <method> [args...]')
.description('Execute Flatfile API commands (e.g., workbooks create, spaces list)')
.option('-k, --apiKey <key>', 'the API Key to use (or set env FLATFILE_API_KEY)')

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nullify Code Language: TypeScript 🟡 HIGH Severity CWE-798

Node api key

A hardcoded API Key is identified. Store it properly in an environment variable.

Reply with /nullify to interact with me like another developer
(you will need to refresh the page for updates)

@flatfile-nullify
Copy link

Nullify Code Vulnerabilities

1 findings found in this pull request

🔴 CRITICAL 🟡 HIGH 🔵 MEDIUM ⚪ LOW
0 1 0 0

You can find a list of all findings here

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