Missing required key 'Bucket' in params.

[trainbolt]

I am getting this error when running gulp now after this integration and I can't seem to locate where it's gone wrong.

/site/web/app/themes/sage/node_modules/gulp-awspublish/node_modules/aws-sdk/lib/param_validator.js:23
throw this.errors[0];
                       ^
MissingRequiredParameter: Missing required key 'Bucket' in params
at fail (/site/web/app/themes/sage/node_modules/gulp-awspublish/node_modules/aws-sdk/lib/param_validator.js:118:37)
at validateStructure (/site/web/app/themes/sage/node_modules/gulp-awspublish/node_modules/aws-sdk/lib/param_validator.js:37:14)
at validateMember (/site/web/app/themes/sage/node_modules/gulp-awspublish/node_modules/aws-sdk/lib/param_validator.js:64:21)
at validate (/site/web/app/themes/sage/node_modules/gulp-awspublish/node_modules/aws-sdk/lib/param_validator.js:12:10)
at Request.VALIDATE_PARAMETERS (/site/web/app/themes/sage/node_modules/gulp-awspublish/node_modules/aws-sdk/lib/event_listeners.js:88:32)
at Request.callListeners (/site/web/app/themes/sage/node_modules/gulp-awspublish/node_modules/aws-sdk/lib/sequential_executor.js:100:18)
at callNextListener (/site/web/app/themes/sage/node_modules/gulp-awspublish/node_modules/aws-sdk/lib/sequential_executor.js:90:14)
at /site/web/app/themes/sage/node_modules/gulp-awspublish/node_modules/aws-sdk/lib/event_listeners.js:75:9
at finish (/site/web/app/themes/sage/node_modules/gulp-awspublish/node_modules/aws-sdk/lib/config.js:228:7)
at /site/web/app/themes/sage/node_modules/gulp-awspublish/node_modules/aws-sdk/lib/config.js:268:9

Here's the gulp/build portion of my gulpfile:

var RevAll = require('gulp-rev-all');
var awspublish = require('gulp-awspublish');
var cloudfront = require("gulp-cloudfront");

var aws = {
    "key": "XXXXXXXXXXXXXXXXXXXX",
    "secret": "XXXXXXXXXXXXXXXXXXXXXXXXXX",
    "bucket": "XXXXXXXX",
    "region": "us-standard",
    "distributionId": "XXXXXXXXXXXX"
};

var publisher = awspublish.create(aws);
var headers = {'Cache-Control': 'max-age=315360000, no-transform, public'};

gulp.task('build', function(callback) {
    runSequence('styles',
                'scripts',
                ['fonts', 'images'],
                callback);
});

gulp.task('default', ['clean'], function() {
    gulp.start('build', function(){
        var revAll = new RevAll();
        gulp.src(path.dist + '**')
            .pipe(revAll.revision())
            .pipe(awspublish.gzip())
            .pipe(publisher.publish(headers))
            .pipe(publisher.cache())
            .pipe(awspublish.reporter())
            .pipe(cloudfront(aws));
    });
});

Everything runs perfectly fine without this.

[trainbolt]

BTW, when I log the params in the param_validator.js I get something like this:

{ Key: 'fonts/FontAwesome.7d8d594e.otf' }

My guess is that somehow that aws object is not getting through?

[smysnk]

As per gulp-awsplugin docs it is expecting the credentials in format:
{
"params": {
"Bucket": "..."
},
"accessKeyId": "...",
"secretAccessKey": "..."
}

I am not sure if it was always like this but the gulp-rev-all documentation needs to be updated.

[trainbolt]

Thanks. I actually had just figured that out as well. They need to be updated for sure. That aws object is not formatted correctly for this.

Also, there's no reference here at all about an upload path... which I'm also currently unsuccessfully implementing.

[smysnk]

For upload path check out their example here:
https://www.npmjs.com/package/gulp-awspublish#rename-file-directory

I would argue this is unrelated to gulp-rev-all plugin besides the example code snippet needing updating.