Skip to content

Conversation

@KrishnaIyer
Copy link
Member

Summary

Create admin API key for development stack

Changes

  • Add option to create an API key for the admin user for dev:initStack.
  • Update doc

Testing

  • Rebuild mage (make init)
  • Run $ tools/bin/mage dev:initStack
  • Copy generated API key and try with CLI.
Regressions

NA

Notes for Reviewers

Run $ make init for this.

Checklist

  • Scope: The referenced issue is addressed, there are no unrelated changes.
  • Compatibility: The changes are backwards compatible with existing API, storage, configuration and CLI, according to the compatibility commitments in README.md for the chosen target branch.
  • Documentation: Relevant documentation is added or updated.
  • Changelog: Significant features, behavior changes, deprecations and fixes are added to CHANGELOG.md.
  • Commits: Commit messages follow guidelines in CONTRIBUTING.md, there are no fixup commits left.

@KrishnaIyer KrishnaIyer self-assigned this Feb 8, 2021
@KrishnaIyer KrishnaIyer added the tooling Development tooling label Feb 8, 2021
@KrishnaIyer KrishnaIyer added this to the February 2021 milestone Feb 8, 2021
Copy link
Contributor

@htdvisser htdvisser left a comment

Choose a reason for hiding this comment

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

To make this more useful, maybe we can our tooling parse that API key and write it to a file in .dev or .env or something?

@KrishnaIyer
Copy link
Member Author

@kschiffer / @asmulko : This PR creates an admin API key and writes that to .env/admin_api_key.txt when you run tools/bin/mage dev:initStack.

This means that you can just use this API for CLI commands during automated testing and not have to login on the CLI.

@KrishnaIyer KrishnaIyer merged commit 724aca2 into v3.11 Feb 22, 2021
@KrishnaIyer KrishnaIyer deleted the feature/dev-api-key branch February 22, 2021 10:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

tooling Development tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants