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

Add fernetenabled rule and unittest #11116

Merged
merged 4 commits into from
Sep 27, 2020

Conversation

Sangarshanan
Copy link
Contributor

@Sangarshanan Sangarshanan commented Sep 24, 2020

Adds FernetEnabled rule to upgrade/rules as per:

https://github.com/apache/airflow/blob/master/UPDATING.md#fernet-is-enabled-by-default

Closes: #11048


Read the Pull Request Guidelines for more information.
In case of fundamental code change, 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 UPDATING.md.

@turbaszek turbaszek linked an issue Sep 24, 2020 that may be closed by this pull request
@turbaszek turbaszek added the area:upgrade Facilitating migration to a newer version of Airflow label Sep 24, 2020
Comment on lines 27 to 30
description = (
"The fernet mechanism is enabled by default "
+ "to increase the security of the default installation."
)
Copy link
Member

@turbaszek turbaszek Sep 24, 2020

Choose a reason for hiding this comment

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

Suggested change
description = (
"The fernet mechanism is enabled by default "
+ "to increase the security of the default installation."
)
description = "The fernet mechanism is enabled by default " \
"to increase the security of the default installation."

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Black did this 😅

@Sangarshanan
Copy link
Contributor Author

Blackened the files, hope the strings are cleaner now

@turbaszek turbaszek merged commit b581362 into apache:v1-10-test Sep 27, 2020
turbaszek pushed a commit to PolideaInternal/airflow that referenced this pull request Oct 21, 2020
turbaszek pushed a commit to PolideaInternal/airflow that referenced this pull request Oct 23, 2020
@potiuk potiuk added this to the Airflow 1.10.13 milestone Nov 14, 2020
cfei18 pushed a commit to cfei18/incubator-airflow that referenced this pull request Mar 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area:upgrade Facilitating migration to a newer version of Airflow
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Create FernetEnabledRule to ease upgrade to Airflow 2.0
3 participants