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.

Sign up for GitHub

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

Jump to bottom

Mask value if the key is token #16474

Merged
merged 2 commits into from Jul 2, 2021
Merged

Mask value if the key is token #16474

merged 2 commits into from Jul 2, 2021

Commits on Jun 17, 2021

  1. Update secrets_masker.py 

    Some connections (including the databricks connection) use the key 'token' in the 'extra' field (this has always been the case). Including it here so that these sensitive tokens are also masked by default.

The prior implementation just masked all of the 'extra' json: "XXXXXXXX" if conn.extra_dejson else None https://github.com/apache/airflow/blob/88199eefccb4c805f8d6527bab5bf600b397c35e/airflow/hooks/base.py#L78
    @robertsaxby
    robertsaxby committed Jun 17, 2021
    Copy the full SHA
    6bf175b View commit details
    Browse the repository at this point in the history

  2. Update secrets_masker.py 

    updated to alphabetical
    @robertsaxby
    robertsaxby committed Jun 17, 2021
    Copy the full SHA
    1c9a11a View commit details
    Browse the repository at this point in the history