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

#242 - Remove Airflow authentication #244

Open
wants to merge 6 commits into
base: develop
Choose a base branch
from

Conversation

nikki-t
Copy link
Collaborator

@nikki-t nikki-t commented Dec 3, 2024

Purpose

Remove Airflow authentication so the user only needs to authenticate with Cognito when navigating to the top-level proxy URL.

Proposed Changes

  • [REMOVED] Removed Airflow authentication

Issues

Testing

  • Deployed modifications on existing Airflow installation and confirmed no authentication was required. No login screen was presented when navigating to the UI and landed on the Airflow dashboard.
  • Destroyed Airflow deployment and re-deployed. Confirmed that no authentication is required for a new deployment.
  • The option to log in is still present in the upper, right of the UI and if the user clicks on the "Log In" button they can log in with the airflow_webserver_username and airflow_webserver_password defined in the deployment's .tfvars file.

https://k8s-sps-airflowi-5934bc27ad-1282880704.us-west-2.elb.amazonaws.com:5000/home
Screenshot 2024-12-03 at 16 14 49

@nikki-t nikki-t self-assigned this Dec 3, 2024
Copy link
Collaborator

@LucaCinquini LucaCinquini left a comment

Choose a reason for hiding this comment

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

I tested that the system behaves exactly as described. We will keep the changes in this branch without merging pending further testing with the CS proxies and discussion with the Unity team.

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.

2 participants