AWS Abstract Client error #3

hellomrdavis · 2017-01-12T15:57:24Z

Slowly but surely we're getting there but no matter what I've tried (several configurations from examples listed on the README page to the sample file provided) I constantly get this error:

`
Notice: Trying to get property of non-object in /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php on line 361

Notice: Trying to get property of non-object in /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php on line 226

Warning: Attempt to assign property of non-object in /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php on line 239

Notice: Trying to get property of non-object in /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php on line 529

Notice: Trying to get property of non-object in /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php on line 530

Notice: Trying to get property of non-object in /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php on line 531

Fatal error: Uncaught TypeError: Argument 1 passed to Aws\Common\Client\AbstractClient::__construct() must be an instance of Aws\Common\Credentials\CredentialsInterface, array given, called in /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php on line 531 and defined in /var/www/html/wp-content/plugins/amazon-web-services/vendor/aws/Aws/Common/Client/AbstractClient.php:73 Stack trace: #0 /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php(531): Aws\Common\Client\AbstractClient->__construct(Array) #1 /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php(374): BCDIAPI->putFileToS3(Object(stdClass)) #2 /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php(251): BCDIAPI->make_request('put_files', Object(stdClass)) #3 /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php(176): BCDIAPI->process_files(Object(stdClass), Object(stdClass)) #4 /var/www/html/wp-content/plugins/ndms-video/ndms-video.php(402): BCDIAPI->ingest_request(Object(stdClass)) #5 /var/www/html/wp-includes/class-wp-ho in /var/www/html/wp-content/plugins/amazon-web-services/vendor/aws/Aws/Common/Client/AbstractClient.php on line 73`

I've checked and my account variables are being passed but it stops at this point. From what I've read I've come across a similar error from someone else plus a proposed solution: http://stackoverflow.com/questions/33594052/s3-client-initialization-error-via-php-sdk

hellomrdavis · 2017-01-13T19:33:34Z

Actually got it to work a few times and then this:

Error:InvalidArgumentException: Credentials must be an instance of Aws\Credentials\CredentialsInterface, an associative array that contains "key", "secret", and an optional "token" key-value pairs, a credentials provider function, or false. in /var/www/html/wp-content/plugins/ndms-video/dist/vendor/Aws/ClientResolver.php:390 Stack trace: #0 /var/www/html/wp-content/plugins/ndms-video/dist/vendor/Aws/ClientResolver.php(267): Aws\ClientResolver::_apply_credentials(Array, Array, Object(Aws\HandlerList)) #1 /var/www/html/wp-content/plugins/ndms-video/dist/vendor/Aws/AwsClient.php(151): Aws\ClientResolver->resolve(Array, Object(Aws\HandlerList)) #2 /var/www/html/wp-content/plugins/ndms-video/dist/vendor/Aws/S3/S3Client.php(214): Aws\AwsClient->__construct(Array) #3 /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php(531): Aws\S3\S3Client->__construct(Array) #4 /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php(373): BCDIAPI->putFileToS3(Object(stdClass)) #5 /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php(250): BCDIAPI->make_request('put_files', Object(stdClass)) #6 /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php(193): BCDIAPI->process_files(Object(stdClass), Object(stdClass)) #7 /var/www/html/wp-content/plugins/ndms-video/ndms-video.php(421): BCDIAPI->ingest_request(Object(stdClass)) #8 /var/www/html/wp-includes/class-wp-hook.php(298): save_ndms_videos_meta(2926, Object(WP_Post), true) #9 /var/www/html/wp-includes/class-wp-hook.php(323): WP_Hook->apply_filters('', Array) #10 /var/www/html/wp-includes/plugin.php(453): WP_Hook->do_action(Array) #11 /var/www/html/wp-includes/post.php(3494): do_action('save_post_ndms_...', 2926, Object(WP_Post), true) #12 /var/www/html/wp-includes/post.php(3578): wp_insert_post(Array, false) #13 /var/www/html/wp-admin/includes/post.php(378): wp_update_post(Array) #14 /var/www/html/wp-admin/post.php(193): edit_post() #15 {main}

Here is the function I created for when a post is saved and it calls the necessary functions, most taken from the sample file:
`$options = get_option('plugin_options');
$videoToken = $options['video_feed_token'];
$videoTag = $options['video_tag'];
$playerPageSize = $options['video_page_size'];
$videoURL = $options['video_url'];
$playerID = $options['player_id'];

$postTitle = get_the_title();

if(isset($_FILES["ndms-video"])){
//$videoFile = get_post_meta( $post->ID, 'ndms-video', true );
$videoDownload = ndms_process_video($post_id, $postTitle);
$videoURL = wp_get_attachment_url($videoDownload);
// $videoURL = parse_url( wp_get_attachment_url($videoDownload) );
// $vurl = dirname( $videoURL [ 'path' ] ) . '/' . rawurlencode( basename( $videoURL[ 'path' ] ) );
}else{
$videoURL = null;
}

// $poster_uploaded_url = "";
// $thumbnail_uploaded_url = "";
// update_post_meta( $post_id, 'ndms-tags', $_POST['ndms-tags'] );

if(isset($_POST["ndms-short-desc"])){
update_post_meta( $post_id, 'ndms-short-desc', $_POST["ndms-short-desc"] );
$videoShortDesc = iconv('ISO-8859-1','UTF-8', $_POST['ndms-short-desc']);
}else{
$videoShortDesc = null;
}

if(isset($_POST["ndms-long-desc"])){
update_post_meta( $post_id, 'ndms-long-desc', $_POST["ndms-long-desc"] );
$videoLongDesc = iconv('ISO-8859-1','UTF-8', $_POST['ndms-long-desc']);
}else{
$videoLongDesc = null;
}

if($_FILES['ndms-thumbnail']['name'] == ''){
$thumbnail_uploaded_url = get_post_meta( $post->ID, 'ndms-thumbnail', true );
}else{
$videoThumbnail = get_post_meta( $post->ID, 'ndms-thumbnail', true );
$thumbnail_uploaded_var = ndms_update_image($post_id,'thumbnail',$postTitle);
// $thumbnail_uploaded_url = parse_url( wp_get_attachment_url($thumbnail_uploaded_var) );
$thumbnail_uploaded_url = wp_get_attachment_url($thumbnail_uploaded_var);
// $thumbUrl = dirname( $thumbnail_uploaded_url[ 'path' ] ) . '/' . rawurlencode( basename( $thumbnail_uploaded_url[ 'path' ] ) );
}

if($_FILES['ndms-poster']['name'] == ''){
$poster_uploaded_url = get_post_meta( $post->ID, 'ndms-poster', true );
}else{
$posterUrl = null;
$videoPoster = get_post_meta( $post->ID, 'ndms-poster', true );
$poster_uploaded_var = ndms_update_image($post_id,'poster',$postTitle);
// $poster_uploaded_url = parse_url( wp_get_attachment_url($poster_uploaded_var) );
$poster_uploaded_url = wp_get_attachment_url($poster_uploaded_var);
// $posterUrl = dirname( $poster_uploaded_url[ 'path' ] ) . '/' . rawurlencode( basename( $poster_uploaded_url[ 'path' ] ) );
}

/**

Push to Brightcove API
*/

// Create Brightcove API Access Token

// Include the BCDIAPI SDK
require 'dist/bc-diapi.php';

// to ingest new video (upload source files)
$video_metadata = '{"name":"'.$postTitle.'","description": "'.$videoShortDesc.'", "long_description" : "'.$videoLongDesc.'", "tags":["'.$videoTag.'"]}';
$push_ingest_data = '{"profile": "videocloud-default-v1","capture-images": false}';
if($videoURL == null && isset($_POST['ndms_video_true'])){
$file_paths = '{"video": "'.$videoURL.'","poster": "'.$poster_uploaded_url.'","thumbnail": "'.$thumbnail_uploaded_url.'"}';
}else{
$file_paths = '{"poster": "'.$poster_uploaded_url.'","thumbnail": "'.$thumbnail_uploaded_url.'"}';
}

// $file_paths = '{"poster": "'.$poster_uploaded_url.'","thumbnail": "'.$thumbnail_uploaded_url.'"}';
$text_tracks = '[{"srclang": "en","kind": "captions","label": "EN","default": true}]';

$push_options = new stdClass();
if(isset($_POST['ndms_video_true'])){
$push_options->video_id = $_POST['ndms_video_true'];
}
$push_options->video_options = $video_metadata;
$push_options->ingest_options = $push_ingest_data;
$push_options->file_paths = $file_paths;
$push_options->text_tracks = $text_tracks;

$account_info = new stdClass();

$options = get_option('plugin_options');
$playerAccount = $options['player_account'];
$bcClientId = $options['brightcove_client_id'];
$bcClientSecret = $options['brightcove_client_secret'];

if (isset($playerAccount)) {
$account_info->account_id = $playerAccount;
} else {
echo 'Account id is required!';
}
if (isset($bcClientId)) {
$account_info->client_id = $bcClientId;
} else {
echo 'Client id is required!';
}
if (isset($bcClientSecret)) {
$account_info->client_secret = $bcClientSecret;
} else {
echo 'Client secret is required!';
}

// account information
$account_data = json_encode($account_info);

// Instantiate the class, passing it the account information.

$bcdi = new BCDIAPI($account_data);

// make a request
// $responses = $bcdi->ingest_request($push_options);

try {
// make request
$responses = $bcdi->ingest_request($push_options);
update_post_meta( $post_id, 'ndms-video-id', $responses->cms->id );
wp_delete_attachment( $videoDownload );
} catch(Exception $error) {
// Handle our error
echo "Error: " . $error;

  die();

}`

rcrooks · 2017-01-13T20:05:04Z

hmmm...I'll have to tinker with this one. Makes no sense to me that it would work several times, then stop. I found the AWS SDK pretty finicky to work with, but once I got it, it seemed to work consistently.

hellomrdavis · 2017-01-13T20:49:21Z

@rcrooks It's definitely been interesting. I found that if I use this in the plugin I'm writing, if I have the AWS Wordpress plugin installed(which is only V2 currently) it overrides the plugin I'm writing with V3 (using your wrapper). Weird. Any help on the above issue would be greatly appreciated.

rcrooks · 2017-01-14T19:17:05Z

@hellomrdavis the wrapper is definitely passing what AWS expects as credentials (this is around line 525 of bc-diapi.php), so ordinarily I would guess that the request for the S3 information (http://docs.brightcove.com/en/video-cloud/di-api/guides/push-based-ingest.html#S3urlRequest) is failing somehow - except that you said this worked a few times and then stopped working -- that's the real puzzler. What are you getting the $responses object that the wrapper returns when you send the request? Anything?

hellomrdavis · 2017-01-17T14:15:57Z

The $responses object is coming back empty. It was definitely weird that it worked perfectly for a moment but then it started popping up errors. If you'd like I could send you a ZIP file of my plugin so you can see the complete codebase. It's attached.

ndms-video.zip

rcrooks · 2017-01-17T14:58:55Z

I'll take a look at the code. It appears none of the requests being sent to wrapper are succeeding, so there must be something....

hellomrdavis · 2017-01-17T16:53:40Z

Great thanks. Look forward to seeing what you find.

rcrooks · 2017-01-17T18:16:34Z

@hellomrdavis just really started digging (nice app btw!) and doubt if this is the main issue, but in noms-video.php lines 370-374 I think you have the cases switched -- if $videoURL == null wouldn't you not want to include the videoURL in file_paths?

hellomrdavis · 2017-01-17T18:24:58Z

Something I've found. It's actually saving to Brightcove but nothing is being passed (video, poster, thumbnail, short description, etc.)

rcrooks · 2017-01-17T19:01:16Z

It's actually encouraging to see something is working! Have a feeling this will turn out to be something in the data you're passing, or the way you're passing it.

hellomrdavis · 2017-01-17T19:23:25Z

I'm feeling that will be the case as well. I actually found the switched cases as well and fixed that. Still didn't fix the issue but it was a good catch.

hellomrdavis · 2017-01-19T15:31:03Z

So I've rewritten some things but still coming up with that same issue. If it helps here's the stack trace that's generated:

Error:InvalidArgumentException: Credentials must be an instance of Aws\Credentials\CredentialsInterface, an associative array that contains "key", "secret", and an optional "token" key-value pairs, a credentials provider function, or false. in /var/www/html/wp-content/plugins/ndms-video/dist/vendor/Aws/ClientResolver.php:390
Stack trace:
#0 /var/www/html/wp-content/plugins/ndms-video/dist/vendor/Aws/ClientResolver.php(267): Aws\ClientResolver::_apply_credentials(Array, Array, Object(Aws\HandlerList))
#1 /var/www/html/wp-content/plugins/ndms-video/dist/vendor/Aws/AwsClient.php(151): Aws\ClientResolver->resolve(Array, Object(Aws\HandlerList))
#2 /var/www/html/wp-content/plugins/ndms-video/dist/vendor/Aws/S3/S3Client.php(214): Aws\AwsClient->__construct(Array)
#3 /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php(531): Aws\S3\S3Client->_construct(Array)
#4 /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php(373): BCDIAPI->putFileToS3(Object(stdClass))
#5 /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php(250): BCDIAPI->make_request('put_files', Object(stdClass))
#6 /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php(193): BCDIAPI->process_files(Object(stdClass), Object(stdClass))
#7 /var/www/html/wp-content/plugins/ndms-video/ndms-video.php(520): BCDIAPI->ingest_request(Object(stdClass))
#8 /var/www/html/wp-includes/class-wp-hook.php(298): save_ndms_videos_meta(3029, Object(WP_Post), true)
#9 /var/www/html/wp-includes/class-wp-hook.php(323): WP_Hook->apply_filters('', Array)
#10 /var/www/html/wp-includes/plugin.php(453): WP_Hook->do_action(Array)
#11 /var/www/html/wp-includes/post.php(3494): do_action('save_post_ndms...', 3029, Object(WP_Post), true)
#12 /var/www/html/wp-includes/post.php(3578): wp_insert_post(Array, false)
#13 /var/www/html/wp-admin/includes/post.php(378): wp_update_post(Array)
#14 /var/www/html/wp-admin/post.php(193): edit_post()
#15 {main}

And...

Notice: Trying to get property of non-object in /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php on line 225

Warning: Attempt to assign property of non-object in /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php on line 233

Notice: Trying to get property of non-object in /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php on line 529

Notice: Trying to get property of non-object in /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php on line 530

Notice: Trying to get property of non-object in /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php on line 531

And here is the final files path being passed in case you see any errors here:
{"video": "/wp-content/uploads/2017/01/Screen-Shot-2017-01-05-at-10.49.30-AM.png","poster": "/wp-content/uploads/2017/01/usa-today-9782284.0-300x200.jpeg","thumbnail": "/wp-content/uploads/2017/01/small-1.mp4"}

hellomrdavis · 2017-01-19T19:38:33Z

So it's fixed once I comment out the "text tracks" snippet. Not sure why that was throwing off the script. Either way we're good to go. However I do have one off topic question. Is it possible to just update metadata through this wrapper? I'm getting the following error if I try to just change metadata and not pass any files:
Notice: Trying to get property of non-object in /var/www/html/wp-content/plugins/ndms-video/dist/bc-diapi.php on line 385

rcrooks · 2017-01-20T15:22:20Z

Hi @hellomrdavis no, it's not possible to just update metadata through this one -- am planning to either add that or create a separate wrapper for pure CMS API operations -- leaning toward the latter, as there are a ton of endpoints for that one. Welcome your input on that though.

rcrooks · 2017-01-20T18:43:09Z

Also, can you tell me what exactly you commented out? Was that in your code or mine? If it was mine, I'd like to track down the issue and fix it.

hellomrdavis · 2017-01-20T20:28:36Z

Yeah it was from your code provided on the README for pushing source video:
$text_tracks = '[{"path": "../assets/sample.vtt", "srclang": "en","kind": "captions","label": "EN","default": true}]';

As far as the metadata I would hate to have to include an additional wrapper purely for passing to the CMS API. Possible to maybe add a switch to the bc-diapi.php file that detects the metadata being passed along with a video ID. Once both are detected it fires a call to the CMS API? Not sure how complex the CMS API is though because you said there are a ton of endpoints.

rcrooks · 2017-01-21T01:04:42Z

Thanks for all that!

hellomrdavis · 2017-01-23T13:48:59Z

No problem. On another note, how intensive would it be to change this wrapper to work with AWS SDK version 2? I'm running into some issues with conflicting SDKs.

rcrooks · 2017-01-24T21:48:12Z

Not sure -- I haven't looked at version 2 at all -- I'll put that on the todo list

hellomrdavis · 2017-02-17T16:32:55Z

@rcrooks Been a while but I've run into an issue that really requires a dynamic ingest wrapper written for v2. Couple questions: 1) do you have a ballpark on how long it would take to do it and 2) how much would you charge to do it?

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

AWS Abstract Client error #3

AWS Abstract Client error #3

hellomrdavis commented Jan 12, 2017

hellomrdavis commented Jan 13, 2017 •

edited

Loading

rcrooks commented Jan 13, 2017

hellomrdavis commented Jan 13, 2017

rcrooks commented Jan 14, 2017

hellomrdavis commented Jan 17, 2017

rcrooks commented Jan 17, 2017

hellomrdavis commented Jan 17, 2017

rcrooks commented Jan 17, 2017

hellomrdavis commented Jan 17, 2017

rcrooks commented Jan 17, 2017

hellomrdavis commented Jan 17, 2017

hellomrdavis commented Jan 19, 2017 •

edited

Loading

hellomrdavis commented Jan 19, 2017 •

edited

Loading

rcrooks commented Jan 20, 2017

rcrooks commented Jan 20, 2017

hellomrdavis commented Jan 20, 2017

rcrooks commented Jan 21, 2017

hellomrdavis commented Jan 23, 2017

rcrooks commented Jan 24, 2017

hellomrdavis commented Feb 17, 2017

AWS Abstract Client error #3

AWS Abstract Client error #3

Comments

hellomrdavis commented Jan 12, 2017

hellomrdavis commented Jan 13, 2017 • edited Loading

rcrooks commented Jan 13, 2017

hellomrdavis commented Jan 13, 2017

rcrooks commented Jan 14, 2017

hellomrdavis commented Jan 17, 2017

rcrooks commented Jan 17, 2017

hellomrdavis commented Jan 17, 2017

rcrooks commented Jan 17, 2017

hellomrdavis commented Jan 17, 2017

rcrooks commented Jan 17, 2017

hellomrdavis commented Jan 17, 2017

hellomrdavis commented Jan 19, 2017 • edited Loading

hellomrdavis commented Jan 19, 2017 • edited Loading

rcrooks commented Jan 20, 2017

rcrooks commented Jan 20, 2017

hellomrdavis commented Jan 20, 2017

rcrooks commented Jan 21, 2017

hellomrdavis commented Jan 23, 2017

rcrooks commented Jan 24, 2017

hellomrdavis commented Feb 17, 2017

hellomrdavis commented Jan 13, 2017 •

edited

Loading

hellomrdavis commented Jan 19, 2017 •

edited

Loading

hellomrdavis commented Jan 19, 2017 •

edited

Loading