Skip to content
This repository has been archived by the owner on Nov 1, 2023. It is now read-only.

add context to all fs calls #423

Merged
6 commits merged into from
Jan 11, 2021
Merged

add context to all fs calls #423

6 commits merged into from
Jan 11, 2021

Conversation

bmc-msft
Copy link
Contributor

@bmc-msft bmc-msft commented Jan 8, 2021

Adds additional context in error handling to all std::fs and tokio::fs calls.

Fixes #309

Copy link
Member

@ranweiler ranweiler left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This will be very useful! Just a few small typos/suggestions noted.

src/agent/onefuzz-agent/src/tasks/stats/afl.rs Outdated Show resolved Hide resolved
src/agent/onefuzz-supervisor/src/setup.rs Outdated Show resolved Hide resolved
src/agent/onefuzz/src/syncdir.rs Outdated Show resolved Hide resolved
@ghost
Copy link

ghost commented Jan 11, 2021

Hello @bmc-msft!

Because this pull request has the auto-merge label, I will be glad to assist with helping to merge this pull request once all check-in policies pass.

p.s. you can customize the way I help with merging this pull request, such as holding this pull request until a specific person approves. Simply @mention me (@msftbot) and give me an instruction to get started! Learn more here.

@ghost ghost merged commit 4657276 into microsoft:main Jan 11, 2021
@bmc-msft bmc-msft deleted the add-context-to-fs branch January 20, 2021 11:12
@ghost ghost locked as resolved and limited conversation to collaborators Apr 17, 2021
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add path context to fallible file accesses
3 participants