Skip to content

Conversation

@omrirh
Copy link
Contributor

@omrirh omrirh commented May 11, 2023

  • Hash resource sensitive data in CRUD methods log messages

@omrirh
Copy link
Contributor Author

omrirh commented May 14, 2023

/verified

@omrirh
Copy link
Contributor Author

omrirh commented May 17, 2023

/verified

@omrirh
Copy link
Contributor Author

omrirh commented May 17, 2023

/verified

myakove
myakove previously approved these changes May 21, 2023
@omrirh
Copy link
Contributor Author

omrirh commented May 21, 2023

/verified

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@omrirh
Copy link
Contributor Author

omrirh commented May 21, 2023

/verified

@myakove myakove merged commit d3aded5 into RedHatQE:main May 21, 2023
@omrirh
Copy link
Contributor Author

omrirh commented May 21, 2023

/cherry-pick v4.13

redhat-qe-bot pushed a commit that referenced this pull request May 21, 2023
* Omit secret data from logs

* use keys attributes hashing to hide sensitive data

* use benedict to modify resource dict

* adding keys_to_hash attribute in resource

* adding hash_log_data attribute to resource object

* using hash_log_data attribute in resource object
@redhat-qe-bot
Copy link
Contributor

Cherry-picked PR Hash sensitive data in resource logs into v4.13

myakove pushed a commit that referenced this pull request May 21, 2023
* Omit secret data from logs

* use keys attributes hashing to hide sensitive data

* use benedict to modify resource dict

* adding keys_to_hash attribute in resource

* adding hash_log_data attribute to resource object

* using hash_log_data attribute in resource object

Co-authored-by: Omri Bar Haim <obarhaim@redhat.com>
@omrirh
Copy link
Contributor Author

omrirh commented May 21, 2023

/cherry-pick v4.12

@redhat-qe-bot
Copy link
Contributor

Manual cherry-pick is needed
Cherry pick failed for d3aded5 to v4.12:
error: could not apply d3aded5... Hash sensitive data in resource logs (#1222)
hint: after resolving the conflicts, mark the corrected paths
hint: with 'git add ' or 'git rm '
hint: and commit the result with 'git commit'

To cherry-pick run:

git fetch --all
git checkout v4.12
git checkout -b d3aded5ce568c6927a9d2c0a5311c8867e0138b
git cherry-pick d3aded5ce568c6927a9d2c0a5311c8867e0138b4
git push origin d3aded5ce568c6927a9d2c0a5311c8867e0138b

omrirh added a commit to omrirh/openshift-python-wrapper that referenced this pull request May 21, 2023
* Omit secret data from logs

* use keys attributes hashing to hide sensitive data

* use benedict to modify resource dict

* adding keys_to_hash attribute in resource

* adding hash_log_data attribute to resource object

* using hash_log_data attribute in resource object
myakove pushed a commit that referenced this pull request May 21, 2023
* Hash sensitive data in resource logs (#1222)

* Omit secret data from logs

* use keys attributes hashing to hide sensitive data

* use benedict to modify resource dict

* adding keys_to_hash attribute in resource

* adding hash_log_data attribute to resource object

* using hash_log_data attribute in resource object

* removing kubernets and python-simple-logger from dependencies
myakove pushed a commit that referenced this pull request Aug 11, 2025
* Omit secret data from logs

* use keys attributes hashing to hide sensitive data

* use benedict to modify resource dict

* adding keys_to_hash attribute in resource

* adding hash_log_data attribute to resource object

* using hash_log_data attribute in resource object
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants