rlauer6 / perl-amazon-s3

A portable client library for working with and managing Amazon S3 buckets and keys.
http://search.cpan.org/dist/Amazon-S3/
2 stars 6 forks source link

Documentation - Mozilla::CA #15

Open vollink opened 9 months ago

vollink commented 9 months ago

Requesting update of the Amazon::S3 POD section, MINIMUM PERL to include Mozilla::CA.

When setting this up for a new deploy, it wouldn't work until Mozilla::CA was also installed. Knowing how build requirements work, I wouldn't expect it to be automatically put in the dependency list by LWP, and I also understand the arguments against doing a require directly from this package's dependency manifest (though, I would advocate for that too, as I'm pretty sure there isn't anything left in AWS that doesn't require functioning SSL). Ultimately, it would be super useful to others if there were a note about this in the documentation as a note under the POD section Minimum Perl (with whatever wording is appropriate about keeping Mozilla::CA up-to-date).

Thanks, Gary

rlauer6 commented 9 months ago

Requesting update of the Amazon::S3 POD section, MINIMUM PERL to include Mozilla::CA.

When setting this up for a new deploy, it wouldn't work until Mozilla::CA was also installed. Knowing how build requirements work, I wouldn't expect it to be automatically put in the dependency list by LWP, and I also understand the arguments against doing a require directly from this package's dependency manifest (though, I would advocate for that too, as I'm pretty sure there isn't anything left in AWS that doesn't require functioning SSL). Ultimately, it would be super useful to others if there were a note about this in the documentation as a note under the POD section Minimum Perl (with whatever wording is appropriate about keeping Mozilla::CA up-to-date).

Thanks, Gary

Thanks for the report. I'll investigate and mitigate if required. What version of Perl were you using? I'm surprised that Mozilla::CA was not automatically installed since CPAN testers have not reported this issue. It may be that their testing environments have that module installed, however many of the testing environments load only core modules when testing a CPAN distribution.

Thanks again for the report.

vollink commented 9 months ago

What version of Perl were you using?

This is perl 5, version 30, subversion 3 (v5.30.3) built for darwin-thread-multi-2level

rlauer6 commented 9 months ago

Sort of explains it ... not many Mac testers :-(