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

Resolve "Authentication for Meltano Analyze" - [merged] #3778

Closed
4 of 5 tasks
MeltyBot opened this issue Jan 28, 2019 · 0 comments
Closed
4 of 5 tasks

Resolve "Authentication for Meltano Analyze" - [merged] #3778

MeltyBot opened this issue Jan 28, 2019 · 0 comments

Comments

@MeltyBot
Copy link
Contributor

MeltyBot commented Jan 28, 2019

Merges 1-authentication-for-meltano-analyze -> master

Migrated from GitLab: https://gitlab.com/meltano/meltano/-/merge_requests/341


Merge Request Checklist

  • Link to an issue
  • Include the proposed fix or feature
  • Include and update tests for the modified code
  • Include a documentation change
  • Add a CHANGELOG.md entry in the Unreleased section

Once your merge request is complete, please assign it a Meltano maintainer for review cycle. Once the review cycle finished, the reviewer shall approve the merge request so it can be merged by any Meltano maintainer.

Closes #1

This MR adds authentication to the Meltano UI using flask-security and flask-authlib.

To enable authentication, you must run meltano ui using the FLASK_ENV=production.
When run in development, authentication will be disabled.

If you want to enable OAuth2 (OpenID Connect) on GitLab.com, you can set the following environment variables in your .env.

Note: the real values can be found in the Meltano 1Password Vault.

.env

OAUTH_GITLAB_APPLICATION_ID=1234567890123456789
OAUTH_GITLAB_SECRET=onetwothreefourfivesix

This MR also adds the first tests to the Meltano API.

References

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

No branches or pull requests

1 participant