Breaking included AWS Phar about phar-stream-wrapper HOT 8 CLOSED

Can you please point to the specific version of the aws.phar file, how it is used/included and mention the exact PHP version (and OS distribution) you're using? Thx

from phar-stream-wrapper.

Hi.
It happens in any Linux version with PHP >= 7.1
I'm using the latest AWS phar file (3.90.3) and requiring it according with the AWS documentation: require '/path/to/aws.phar';

from phar-stream-wrapper.

Current workaround

instead of

require('/path/to/aws.phar');

please use (for the time being)

require('phar:///path/to/aws.phar/aws-autoloader.php');

This is what the stub of aws.phar is doing anyway internally.

Explanation

The stub of libraries like geoip2 (build with php-box) looks like the following. When invoking the phar:// stream, the full path is still given internally by using __FILE__. That way, the file path can be mapped to its internal alias name.

Phar::mapPhar('geoip2.phar');
require 'phar://' . __FILE__ . '/phar-stub.php';

The stub of the aws library looks like the following and does not contain any internal reference to it's original file. That's the reason why the internal alias aws-3.90.3.phar cannot be checked and results in an empty base-name. The work-around above uses the phar:// stream and thus triggers alias resolving.

Phar::mapPhar('aws-3.90.3.phar');
require 'phar://aws-3.90.3.phar/aws-autoloader.php';

from phar-stream-wrapper.

In PR #22 I've added test cases first in order to reproduce the behavior. Next step would be a fix...

from phar-stream-wrapper.

I needed only require 'phar://aws-3.90.3.phar/aws-autoloader.php'; and it worked again.

from phar-stream-wrapper.

@ohader this solution does not work with "geoip2.phar"?

from phar-stream-wrapper.

i have workaround for "GeoIP2.phar"
require_once ( 'phar://' . __DIR__ . '/geoip2.phar/vendor/autoload.php' );

from phar-stream-wrapper.

Please test the references pull-requests #31 (PHP7) or #32 (PHP5) and check whether the originally reported bug of this ticket is solved. Thanks!

from phar-stream-wrapper.