Page MenuHomePhabricator

[Identity] Add new Error variant
ClosedPublic

Authored by varun on May 6 2022, 1:26 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Dec 20, 10:53 PM
Unknown Object (File)
Fri, Dec 20, 10:53 PM
Unknown Object (File)
Fri, Dec 20, 10:53 PM
Unknown Object (File)
Fri, Dec 20, 10:51 PM
Unknown Object (File)
Sat, Nov 30, 4:52 AM
Unknown Object (File)
Tue, Nov 26, 7:57 PM
Unknown Object (File)
Nov 23 2024, 2:33 PM
Unknown Object (File)
Nov 23 2024, 10:09 AM

Details

Summary

The Rusoto Error variant is too specific to handle both get and put errors. I'm therefore breaking it out into two variants, RusotoGet and RusotoPut. This will help us handle errors related to the put_item Rusoto method.

Depends on D3960

Test Plan

cargo build

Diff Detail

Repository
rCOMM Comm
Branch
varun/identity_service (branched from master)
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

varun requested review of this revision.May 6 2022, 1:33 PM
This revision is now accepted and ready to land.May 9 2022, 3:25 AM

I'm not sure why this is in an "accepted" state. I hope it cannot be landed... After I accept, rust diffs should not be accepted, I know rust far too little to be decisive. Adding Jim as blocking.

This revision now requires review to proceed.May 9 2022, 4:53 AM
This revision is now accepted and ready to land.May 9 2022, 10:08 AM
This revision was automatically updated to reflect the committed changes.