You are correct. When porting the changes back (we just create a s3_file
provider which extends remote_file and install it directly to chef) I
forgot that bucket was a new reference. I'll change it when I get to q PC.
Did you encounter any other errors if that was corrected?
On Apr 16, 2014 5:54 PM, "Brandon Adams" <notifications@github.com> wrote:

> In providers/default.rb:
>
> > @@ -32,15 +33,30 @@
> >    end
> >
> >    if ::File.exists?(new_resource.path)
> > -    s3_md5 = S3FileLib::get_md5_from_s3(new_resource.bucket, remote_path, aws_access_key_id, aws_secret_access_key, token)
> > -    local_md5 = Digest::MD5.hexdigest(::File.read(new_resource.path))
> > +    if decryption_key.nil?
> > +      if new_resource.decrypted_file_checksum.nil?
> > +        s3_md5 = S3FileLib::get_md5_from_s3(bucket, remote_path, access_key_id, secret_access_key, token)
>
> Line 38 of providers/default.rb is throwing errors in my testing
> environment. It looks like the arguments are a little off. They should be
> (new_resource.bucket, remote_path, aws_access_key_id,
> aws_secret_access_key, token).
>
> --
> Reply to this email directly or view it on GitHub<https://github.com/adamsb6/s3_file/pull/25/files#r11709629>
> .
>