Gaufrette
Gaufrette copied to clipboard
Remove Gaufrette\Adapter\AwsS3 capability of silence exceptions
I really don't know if there's some BC reason to those "return null" (i.e. exceptions catch in read() and write() methods). For any reason if there's something wrong with credentials (signature) there will be no hint at runtime (a colorful "null" at most), this is really bad.
bump?
Is there a way to find or display those errors currently ?
+1 to this. It makes debugging S3 server responsive unnecessarily difficult. I'd be happy to provide a pull request if there isn't a reason to catch these exceptions.
Unfortunately, the Adapter interface currently specify that it must return false on failure instead of throwing exception (which is totally bad design, I agree). changing this is a BC break though.
A full rewrite of Gaufrette has started on https://github.com/Gaufrette/ though
Thanks for the info. Glad to hear this is being addressed in the next version.