screwdriver icon indicating copy to clipboard operation
screwdriver copied to clipboard

Remove `catbox-s3` dependency of store

Open jithine opened this issue 4 years ago • 2 comments

What happened:

Store uses catbox-s3 for talking to AWS. Due to it's limitation we removed it for build artifacts, but it's still used for caches end point.

What you expected to happen:

AWS Client should be directly used for any s3 related use cases, instead of catbox-s3.

How to reproduce it:

jithine avatar Jun 08 '21 23:06 jithine

Hi, creator of catbox-s3 here. First of all, thanks, I wasn't aware that you are using my library. I haven't been working with hapi/catbox for a long time and just wanted to point out that the last release of catbox-s3 is already four years old. I already flagged it as 'deprecated' on npm, as I don't intend to release further updates for it.

So I'd recommend to either remove it as dependency in future versions, or let me know if you're interested in maintaining it.

fhemberger avatar Mar 21 '22 16:03 fhemberger

Hello @fhemberger Thanks for the update. We are looking to usie native aws sdk for S3 connections. So we will be removing the dependency once we get to this.

jithine avatar Mar 21 '22 16:03 jithine