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

fix: logging whole resource just on trace level #1985

Merged
merged 2 commits into from
Jul 31, 2023
Merged

fix: logging whole resource just on trace level #1985

merged 2 commits into from
Jul 31, 2023

Conversation

csviri
Copy link
Collaborator

@csviri csviri commented Jul 28, 2023

In case of Secrets this migh log sensitice data.

In case of Secrets this migh log sensitice data.
@csviri csviri self-assigned this Jul 28, 2023
@openshift-ci openshift-ci bot requested review from adam-sandor and andreaTP July 28, 2023 08:30
@vmuzikar
Copy link
Contributor

Related discussion #1983

@csviri csviri requested a review from metacosm July 31, 2023 07:23
@csviri csviri merged commit 03e06ce into main Jul 31, 2023
@csviri csviri deleted the trace-dr-log branch July 31, 2023 08:01
@vmuzikar
Copy link
Contributor

vmuzikar commented Aug 7, 2023

Will this get backported to JOSDK 4 and to QOSDK 6? We'd like to have this in Quarkus 3.2.

@csviri
Copy link
Collaborator Author

csviri commented Aug 7, 2023

@vmuzikar yes, it will be.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants