Skip to content

Conversation

@amnguye
Copy link
Member

@amnguye amnguye commented Oct 23, 2025

Attempt to address: #49993

In order to ensure polling and scanning operations use the "target" BlobServiceClient specified we had to make changes to how Listeners are added and the BlobListenerStrategy is initialized.

In my attempt here, I did the following:

  • Changed IBlobNotificationStrategy to now take in a "target" BlobServiceClient
    • This updated all the derived *Strategy's as well
  • Updated the PollLogsStrategy check on if "$logs" is enabled on the Blob Storage Account
    • I added a try-catch here because ScanBlobScanLogHybridPollingStrategy will eventually call this down the line to prevent unncessary double calls to GetProperties and SetProperties.
  • Fall back to the primary blob storage account
    • This was done to maintain behavior in the case that users want the primary account to be used

TODO:

  • Write up unit tests
  • Ensure the error code being caught for permissions check is correct, to fall back to primary account

…y.RegisterAsync; Added catch for possible permissions failure to default to primary
@github-actions github-actions bot added the Storage Storage Service (Queues, Blobs, Files) label Oct 23, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Storage Storage Service (Queues, Blobs, Files)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant