ReviewOS

also looking at this

meemalabs/laravel-media-converter

Read account url from config instead of sending a http request to AWS…

#10
Merged DennisOng wants to merge account-endpoint-via-config into main
3 files +4 -4

Review threads live on the whole diff, not on one commit, so none are shown here - a thread's line means something in the branch's final form, and painting it into an intermediate step would put it on code it is not about.

README.mdmodified+2-2
Changes to README.md
@@ -50,7 +50,7 @@ Next, please add the following keys their values to your `.env` file.
5050AWS_ACCESS_KEY_ID=xxxxxxx
5151AWS_SECRET_ACCESS_KEY=xxxxxxx
5252AWS_DEFAULT_REGION=us-east-1
53AWS_MEDIA_CONVERT_ACCOUNT_URL=https://xxxxxxxxx.mediaconvert.us-east-1.amazonaws.com
53AWS_MEDIACONVERT_ACCOUNT_URL=https://xxxxxxxxx.mediaconvert.us-east-1.amazonaws.com
5454AWS_IAM_ARN=arn:aws:iam::xxxxxxx:role/MediaConvert_Default_Role
5555AWS_QUEUE_ARN=arn:aws:mediaconvert:us-east-1:xxxxxxx:queues/Default
5656```
@@ -73,7 +73,7 @@ return [
7373 ],
7474 'region' => env('AWS_DEFAULT_REGION', 'us-east-1'),
7575 'version' => 'latest',
76 'url' => env('AWS_MEDIA_CONVERT_ACCOUNT_URL'),
76 'url' => env('AWS_MEDIACONVERT_ACCOUNT_URL'),
7777
7878 /**
7979 * Specify the IAM Role ARN.
config/config.phpmodified+1-1
Changes to config/config.php
@@ -15,7 +15,7 @@ return [
1515 ],
1616 'region' => env('AWS_DEFAULT_REGION', 'us-east-1'),
1717 'version' => 'latest',
18 'url' => env('AWS_MEDIA_CONVERT_ACCOUNT_URL'),
18 'url' => env('AWS_MEDIACONVERT_ACCOUNT_URL'),
1919
2020 /**
2121 * Specify the IAM Role ARN.