Skip to content

Conversation

@lwyszomi
Copy link
Contributor

@lwyszomi lwyszomi commented Mar 8, 2022

Fixed vulnerable issue with using yaml load


^ Add meaningful description above

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.

@lwyszomi lwyszomi requested a review from turbaszek as a code owner March 8, 2022 14:19
@boring-cyborg boring-cyborg bot added area:providers provider:google Google (including GCP) related issues labels Mar 8, 2022
@potiuk potiuk merged commit 7f4935b into apache:main Mar 8, 2022
@lwyszomi
Copy link
Contributor Author

lwyszomi commented Mar 8, 2022

@potiuk ups, tests failed, look real

@potiuk
Copy link
Member

potiuk commented Mar 8, 2022

Ups. Too fast

potiuk added a commit to potiuk/airflow that referenced this pull request Mar 8, 2022
@lwyszomi
Copy link
Contributor Author

lwyszomi commented Mar 8, 2022

I will check this once again, I didn't know that safe_load_all return generator 😮

@potiuk
Copy link
Member

potiuk commented Mar 8, 2022

Yep. Revert in progress.

potiuk added a commit that referenced this pull request Mar 8, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:providers provider:google Google (including GCP) related issues

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants