Skip to content

Commit

Permalink
Fix deprecated jwt decode method (#919)
Browse files Browse the repository at this point in the history
Later versions of airflow removed this function on us, but looks like we can just use a function bound to the jwt library itself directly.
  • Loading branch information
michaeljguarino authored Sep 27, 2024
1 parent d0705d2 commit d1ed04f
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.42
version: 0.3.43
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 @@ -68,7 +68,7 @@ airflow:
class PluralSecurityManager(AirflowSecurityManager):
def _get_oauth_user_info(self, provider, response):
if provider == "plural":
me = self._azure_jwt_token_parse(response["id_token"])
me = jwt.decode(response["id_token"], options={"verify_signature": False})
split_name = me["name"].split()
return {
"username": me["name"],
Expand Down

0 comments on commit d1ed04f

Please sign in to comment.