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

Mark sensitive fields as Sensitive #816

Merged
merged 1 commit into from
Jan 27, 2021
Merged

Mark sensitive fields as Sensitive #816

merged 1 commit into from
Jan 27, 2021

Conversation

outcatcher
Copy link
Contributor

Summary of the Pull Request

Set Sensitive: true for sensitive fields.
Fix #815

PR Checklist

@outcatcher outcatcher requested review from a team, lego963 and anton-sidelnikov and removed request for a team January 27, 2021 14:22
@outcatcher outcatcher added the gate Merge PR label Jan 27, 2021
@otc-zuul
Copy link

otc-zuul bot commented Jan 27, 2021

Build succeeded.

@otc-zuul otc-zuul bot merged commit 0a2eacf into master Jan 27, 2021
@otc-zuul otc-zuul bot deleted the sensitive branch January 27, 2021 14:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@anton-sidelnikov anton-sidelnikov anton-sidelnikov approved these changes

@otc-zuul otc-zuul[bot] otc-zuul[bot] approved these changes

@lego963 lego963 lego963 approved these changes

Assignees
No one assigned
Labels
gate Merge PR
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Some provider fields are not marked as sensitive
3 participants
@outcatcher @lego963 @anton-sidelnikov