[backup] Remove redundant blob client auth
ClosedPublic
Actions

Authored by bartek on May 12 2025, 9:17 AM.

Details

Reviewers

kamil

Commits

rCOMMb62017b8b437: [backup] Remove redundant blob client auth

Summary

These lines are redundant because Authenticated<BlobServiceClient> does this for us.

Depends on D14682

Test Plan

cargo build, backup integration tests

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

bartek created this revision.May 12 2025, 9:17 AM

bartek held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptMay 12 2025, 9:17 AM

Harbormaster completed remote builds in B34145: Diff 48032.May 12 2025, 9:39 AM

bartek published this revision for review.May 12 2025, 10:15 AM

bartek added a child revision: D14684: [backup] Create function to copy logs between backups.

kamil accepted this revision.May 14 2025, 11:02 AM

This revision is now accepted and ready to land.May 14 2025, 11:02 AM

Closed by commit rCOMMb62017b8b437: [backup] Remove redundant blob client auth. · Explain WhyMay 16 2025, 1:35 PM

This revision was automatically updated to reflect the committed changes.

bartek added a commit: rCOMMb62017b8b437: [backup] Remove redundant blob client auth.

Revision Contents
Changeset List

Path

Size

services/

backup/

src/

http/

handlers/

backup.rs

4 lines

Diff 48106

View Options

services/backup/src/http/handlers/backup.rs

[backup] Remove redundant blob client authClosedPublicActions