Skip to content

Commit 47d805e

Browse files
committed
Merge pull request #84 from chuck014/master
remove credentials if blank, causing the SDK to use ec2 role
2 parents 02adcec + 744ebd0 commit 47d805e

File tree

3 files changed

+7
-5
lines changed

3 files changed

+7
-5
lines changed

.travis.yml

+4
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,10 @@ php:
88

99
sudo: false
1010

11+
matrix:
12+
allow_failures:
13+
- php: hhvm
14+
1115
install: travis_retry composer install --no-interaction --prefer-source
1216

1317
script: vendor/bin/phpunit

config/aws.php

+1-4
Original file line numberDiff line numberDiff line change
@@ -16,10 +16,7 @@
1616
| http://docs.aws.amazon.com/aws-sdk-php/v3/guide/guide/configuration.html
1717
|
1818
*/
19-
'credentials' => [
20-
'key' => env('AWS_ACCESS_KEY_ID'),
21-
'secret' => env('AWS_SECRET_ACCESS_KEY'),
22-
],
19+
2320
'region' => env('AWS_REGION', 'us-east-1'),
2421
'version' => 'latest',
2522
'ua_append' => [

src/AwsServiceProvider.php

+2-1
Original file line numberDiff line numberDiff line change
@@ -41,7 +41,8 @@ public function boot()
4141
public function register()
4242
{
4343
$this->app->singleton('aws', function ($app) {
44-
return new Sdk($app['config']->get('aws'));
44+
$config = $app['config']->get('aws');
45+
return new Sdk($config);
4546
});
4647

4748
$this->app->alias('aws', 'Aws\Sdk');

0 commit comments

Comments
 (0)