Skip to content

Conversation

@suzmue
Copy link
Collaborator

@suzmue suzmue commented Aug 22, 2025

Fixes #2975, fixes #2976, fixes #2977, and fixes #3042

@suzmue suzmue requested a review from a team as a code owner August 22, 2025 21:55
@product-auto-label product-auto-label bot added the api: storage Issues related to the Cloud Storage API. label Aug 22, 2025
@codecov
Copy link

codecov bot commented Aug 22, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 96.49%. Comparing base (14137a3) to head (ea6d80d).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3004      +/-   ##
==========================================
- Coverage   96.51%   96.49%   -0.03%     
==========================================
  Files         107      107              
  Lines        4396     4398       +2     
==========================================
+ Hits         4243     4244       +1     
- Misses        153      154       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link
Collaborator

@coryan coryan left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

A few nits and thoughts for you to consider.

I also updated the description, you need a fixes #blah for each blah.

@suzmue suzmue force-pushed the encrypted_storage_samples branch from dfc5d1c to 1196971 Compare August 25, 2025 17:33
@suzmue
Copy link
Collaborator Author

suzmue commented Aug 25, 2025

@coryan Making the changes you suggested revealed a couple of places that make KeyAes256 hard to use. I ended up implementing Clone and Display traits for the type as well.

@suzmue suzmue changed the title docs(storage): add examples for using CSEK feat(storage): implement Clone and Display for KeyAes256 and add examples for using CSEK Aug 25, 2025
@suzmue suzmue merged commit 3b6b34a into googleapis:main Aug 25, 2025
24 checks passed
@suzmue suzmue deleted the encrypted_storage_samples branch August 25, 2025 19:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

api: storage Issues related to the Cloud Storage API.

Projects

None yet

2 participants