Skip to content

Ensure that we generate a jwt_secret before launching api server processes#47739

Merged
ashb merged 1 commit into
apache:mainfrom
astronomer:generate-signing-key-in-parent
Mar 13, 2025
Merged

Ensure that we generate a jwt_secret before launching api server processes#47739
ashb merged 1 commit into
apache:mainfrom
astronomer:generate-signing-key-in-parent

Conversation

@ashb

@ashb ashb commented Mar 13, 2025

Copy link
Copy Markdown
Member

Otherwise each individual process will generate its own which results in n
different secret keys in use, and leads to random 403 errors.


^ Add meaningful description above
Read the Pull Request Guidelines for more information.
In case of fundamental code changes, an Airflow Improvement Proposal (AIP) is needed.
In case of a new dependency, check compliance with the ASF 3rd Party License Policy.
In case of backwards incompatible changes please leave a note in a newsfragment file, named {pr_number}.significant.rst or {issue_number}.significant.rst, in newsfragments.

…cesses

Otherwise each individual process will generate its own which results in _n_
different secret keys in use, and leads to random 403 errors.
@ashb ashb changed the title Ensure that we generate a jwt_secret before launching api servers processes Ensure that we generate a jwt_secret before launching api server processes Mar 13, 2025

@potiuk potiuk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

nice!

@ashb ashb merged commit 4d279e7 into apache:main Mar 13, 2025
@ashb ashb deleted the generate-signing-key-in-parent branch March 13, 2025 16:40
nailo2c pushed a commit to nailo2c/airflow that referenced this pull request Apr 4, 2025
…cesses (apache#47739)

Otherwise each individual process will generate its own which results in _n_
different secret keys in use, and leads to random 403 errors.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants