Skip to content

Commit

Permalink
Fix deprecated jwt decode method
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 committed Sep 27, 2024
1 parent d0705d2 commit 894bd8a
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 894bd8a

Please sign in to comment.