Skip to content

Conversation

@ercsnmrs
Copy link
Contributor

Overview:

  • KMS-protected buckets gives full control over encryption keys (vs Google-managed keys), granular permissions: You can restrict access to specific keys per team or service, and rotation & auditability: You can rotate keys periodically and audit access using Cloud Logging.

Overview:
- KMS-protected buckets gives full control over encryption keys (vs Google-managed keys), granular permissions: You can restrict access to specific keys per team or service, and rotation & auditability: You can rotate keys periodically and audit access using Cloud Logging.
@ercsnmrs
Copy link
Contributor Author

implemented the same approach on an enterprise solution

  • I will test it out first by removing the limitation on maxInbandAttachmentSize on the Webapp
  • I will post the results here.

@or-else
Copy link
Contributor

or-else commented Aug 31, 2025

Cool, thanks.

removing the limitation on maxInbandAttachmentSize on the Webapp

I'm not sure what you mean by this. Do you want to set maxInbandAttachmentSize to zero?

@ercsnmrs
Copy link
Contributor Author

ercsnmrs commented Aug 31, 2025

@or-else i am trying to test the upper limit of the integration (e.g rps on which the integration to kms buckets break).

One use case i see here is that we can offload all images and attachment to the cloud instead of saving to db. I see it as a path for optimization here.

Todo this, we need to send everything to gcs by lowering the maxInBandAttachmensize or disabling (i will do this locally)

@or-else
Copy link
Contributor

or-else commented Aug 31, 2025

OK, makes sense.

@or-else
Copy link
Contributor

or-else commented Sep 1, 2025

Should I review this PR now too?

@or-else
Copy link
Contributor

or-else commented Sep 19, 2025

Any update on this?

@ercsnmrs
Copy link
Contributor Author

@or-else will update tomorrow. thanks!

@or-else
Copy link
Contributor

or-else commented Nov 3, 2025

Any update on this?

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