gazette / core

Build platforms that flexibly mix SQL, batch, and stream processing paradigms
https://gazette.dev
MIT License
718 stars 52 forks source link

fix: Azure fragment store with multiple tenants #354

Closed jshearer closed 11 months ago

jshearer commented 11 months ago

I knew that the service credentials would be scoped to tenant IDs, but I did not realize that the service.Clients and UserDelegationCredentials would also be tenant-scoped. This was causing problems where the credentials of whatever tenant was first seen would be used to sign all SignGet URLs, resulting in signature mismatch errors when the blob was owned by a different tenant.

This updates the cache logic to keep track of both the service clients and the user delegation credentials by tenant ID, which should solve the problem.


This change is Reviewable