search

baopham / laravel-dynamodb

Eloquent syntax for DynamoDB
https://packagist.org/packages/baopham/dynamodb
MIT License
484 stars 128 forks source link

Assume role support #260

Closed adenot closed 1 year ago

adenot commented 1 year ago

I've added assume role support as we need to fetch data from a dynamodb in a different AWS account.

scrutinizer-notifier commented 1 year ago

The inspection completed: 1 new issues

sapher commented 1 year ago

I need that, hopes it get merged

nelson6e65 commented 1 year ago

@adenot @sapher For the moment I have not finished the base tests to check this.

I will merge it under role-support branch provisionally, so you can install as dev-role-support version to be able to test it manually in your composer.json.

  "require": {    
      "baopham/dynamodb": "dev-role-support"
  },
composer update baopham/dynamodb

@adenot @sapher @zgrguric Can you test this manually and give me some feedback if something is wrong or works fine? 🥺 I'm not using this in my current projects, so I can't test it by myself in the short time. 😅

nelson6e65 commented 1 year ago

@zgrguric https://github.com/baopham/laravel-dynamodb/pull/260#issuecomment-1512466617