-
Notifications
You must be signed in to change notification settings - Fork 305
Is this library still supported? #129
Comments
😞 |
I made a fundamental mistake when making this library, which was depending on the official AWS SDK package, which is probably the worst piece of shit JavaScript package I've ever depended on. I put so much careful thought into preventing race conditions and handling every situation correctly and efficiently, only to have my effort wasted by incompetent Amazon engineers. I should have known better, I worked there for a few months before NOPEing the hell out of there. Look at how much bullshit I've had to wade through:
aws/aws-sdk-js#291 So if this module is going to survive, it must be changed to remove the dependency on the aws sdk js package and instead make direct HTTP requests, like knox already does. I'm over here working on Genesis DAW and Zig language, not really motivated to put work into this Node.js library unless someone wants to pay me by the hour. If someone proves to be competent and wants to take over maintenance of this module I will happily hand it over. |
Thank you for your answer! |
@andrewrk I'm sorry to hear of all your issues with the Amazon library. For what it matters, I've updated the embedded AWS-SDK to latest AWS-SDK (2.3.x) and your library still works. Your library is very useful and makes it easy to work with S3, especially as knox (without another version) doesn't support multipart-upload. |
@andrewrk, @thenitai knox also doesn't appear to support 'sync-like' functionality, which you do a great-job with via Yours is the best of the related libraries we've looked at:
Additionally, sync is not included in the SDK. CLI has many known issues:
Does anyone have a recommendation for a sync library that I've missed? One that doesn't drop files without warning (thanks aws-sdk 😿) Recommendations on syncing with per-file metadata? |
+1 |
@andrewrk Would you be amenable to adding a "Needs Maintainer" message on the readme and repo description, potentially linking to this issue from the readme? You can figure it out based on the last commit date, open issues, etc., but the explicit call out would be nice. |
Hi!
I see there are quite a lot of (unresolved) open issues and forgotten pull requests, which makes me wonder if this library is still supported?
If not, it would be a good idea to mention it in the read me.
Thanks :)
Christophe
The text was updated successfully, but these errors were encountered: