Skip to content

Commit

Permalink
Fix airflow fab security manager
Browse files Browse the repository at this point in the history
Need to use the airflow in-tree import not from flask appbuilder itself for later versions
  • Loading branch information
michaeljguarino committed Feb 6, 2024
1 parent 8f95f18 commit 9efba8c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
2 changes: 1 addition & 1 deletion airflow/helm/airflow/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: airflow
description: A Helm chart for airflow deployable on plural
type: application
version: 0.3.39
version: 0.3.40
appVersion: "2.6.3"
sources:
- https://github.com/pluralsh/plural-artifacts/airflow/helm/airflow
Expand Down
2 changes: 1 addition & 1 deletion airflow/helm/airflow/values.yaml.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -63,7 +63,7 @@ airflow:
import jwt
from airflow import configuration as conf
from airflow.www.security import AirflowSecurityManager
from flask_appbuilder.security.manager import AUTH_OAUTH
from airflow.www.fab_security.manager import AUTH_OAUTH

class PluralSecurityManager(AirflowSecurityManager):
def _get_oauth_user_info(self, provider, response):
Expand Down

0 comments on commit 9efba8c

Please sign in to comment.