phar-updater icon indicating copy to clipboard operation
phar-updater copied to clipboard

a problem in a dependend library causes the github strategy to fail on PHP versions < 5.6.0

Open ppetermann opened this issue 10 years ago • 7 comments

the issue described in: https://github.com/padraic/file_get_contents/issues/4

causes the github strategy (on PHP < 5.6.0) to fail with: file_get_contents(): Peer certificate CN=s3.amazonaws.com' did not match expected CN=github.com'

ppetermann avatar Jun 04 '15 17:06 ppetermann

Also having same problem. I see this haven't been fixes for almost 1 year now.

aik099 avatar May 29 '16 14:05 aik099

I'm wondering why curl with same SSL peer verification (CURLOPT_SSL_VERIFYHOST option) can't be used instead of humbug_get_contents function/package.

aik099 avatar May 29 '16 14:05 aik099

Here is what I've did in my library fork to get this working: https://github.com/console-helpers/phar-updater/commit/6dc6cdad0fe05c7d0a455017e44c636236c6d4de

aik099 avatar May 29 '16 15:05 aik099

Still needed?

Sorry for the delay, been AWOL.

padraic avatar Mar 29 '17 14:03 padraic

Sure. But this original issue that needs fixing is https://github.com/padraic/file_get_contents/issues/4. This issue is just a link so that we'll know that what's happening in here is result of an issue in a dependency.

aik099 avatar Mar 29 '17 15:03 aik099

on a side note, would it make sense to move https://github.com/padraic/file_get_contents under the Humbug umbrella?

theofidry avatar Apr 14 '17 20:04 theofidry

@theofidry Absolutely. I also have the github site for the PHAR downloads.

Edit: Transferred the file_get_contents repo and updated the Packagist site.

padraic avatar Apr 14 '17 20:04 padraic