Page MenuHomePhabricator

[services] Backup - prevent sending logs for non-existing backups
ClosedPublic

Authored by karol on May 19 2022, 3:00 AM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Sep 4, 6:46 AM
Unknown Object (File)
Tue, Sep 3, 5:26 AM
Unknown Object (File)
Tue, Sep 3, 5:26 AM
Unknown Object (File)
Tue, Sep 3, 5:25 AM
Unknown Object (File)
Tue, Sep 3, 5:20 AM
Unknown Object (File)
Aug 27 2024, 2:36 PM
Unknown Object (File)
Aug 26 2024, 5:08 AM
Unknown Object (File)
Aug 2 2024, 4:58 PM

Details

Summary

Depends on D4074

This is a bug fix. If the backup for the given backup id does not exist in the database, we should abort sending log.

Test Plan
  • terminal 1
cd services
yarn run-blob-service-dev-mode
  • terminal 2
cd services
yarn run-backup-service-dev-mode
  • terminal 3
git clone https://github.com/karol-bisztyga/grpc-playground.git
cd grpc-playground
git checkout backup-async

You can hard-code some fake backup-id in the code (Client.h - lastBackupID)

./build.sh
./cmake/build/bin/client

Then in terminal 3 create a new log

l

With an invalid current backup id, the log cannot be sent.

Diff Detail

Repository
rCOMM Comm
Lint
No Lint Coverage
Unit
No Test Coverage