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

Use DEFAULT_ADMIN_EMAIL everywhere to prevent legacy key usage warnings on fresh installs #1509

Merged
merged 1 commit into from
Mar 13, 2023

Conversation

AndrewDryga
Copy link
Collaborator

No description provided.

```
docker compose run --rm firezone bin/migrate
```

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is VSCode markdown formatter, should I roll it back?

Copy link
Member

Choose a reason for hiding this comment

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

No worries, if it passes CI it should be fine

@coveralls
Copy link

Pull Request Test Coverage Report for Build b03921a8121d2ceee2dd33a931478258ae4617b3-PR-1509

  • 2 of 2 (100.0%) changed or added relevant lines in 1 file are covered.
  • No unchanged relevant lines lost coverage.
  • Overall coverage remained the same at 66.904%

Totals Coverage Status
Change from base Build a34fafcae792aa603a27636a6e4f82d43074223b: 0.0%
Covered Lines: 1690
Relevant Lines: 2526

💛 - Coveralls

@AndrewDryga AndrewDryga enabled auto-merge (squash) March 13, 2023 21:22
@AndrewDryga AndrewDryga merged commit 6eba824 into master Mar 13, 2023
@AndrewDryga AndrewDryga deleted the andrew/use-default-admin-email-env branch March 13, 2023 21:33
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.

None yet

3 participants