This repository was archived by the owner on Dec 7, 2022. It is now read-only.

[Feature request] Map AccessDenied S3 error to PermissionMissing #130

Open

jrbarnard opened this issue Mar 4, 2020 · 1 comment

Labels

🚀 Enhancement 👨‍💻 PR Needed

jrbarnard commented Mar 4, 2020

Currently an AccessDenied comes back as an UnknownException.

This would be good as a PermissionMissing generic error similar to local & GCP drivers.

Happy to do a PR for this if there is interest

Member

RomainLanz commented May 19, 2020

Hey @jrbarnard! 👋

We are happily accepting PR!

However, I'd recommend to wait until we finish to swap the architecture.
I'll let you know.

RomainLanz added the 🚀 Enhancement label

RomainLanz added the 👨‍💻 PR Needed label

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.