Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Type s3 errors #42

Merged
merged 3 commits into from
Mar 15, 2024
Merged

feat: Type s3 errors #42

merged 3 commits into from
Mar 15, 2024

Conversation

godu
Copy link
Contributor

@godu godu commented Mar 13, 2024

I needed NoSuchBucket exception on HeadBucket.

(I used codegen, that explains the difference)

@godu godu force-pushed the client-s3-errors branch from 391dd0d to df433a4 Compare March 13, 2024 16:53
@floydspace
Copy link
Owner

Apparently aws defined these errors recently, at the time I implemented s3 client there was only one generic error, it's good that we have codegen now 👍🏼

@godu
Copy link
Contributor Author

godu commented Mar 14, 2024

@floydspace we may need this one before #43

@floydspace floydspace merged commit 2fd893a into floydspace:main Mar 15, 2024
3 checks passed
@github-actions github-actions bot mentioned this pull request Mar 15, 2024
@godu godu deleted the client-s3-errors branch April 24, 2024 08:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants