Turistforeningen / node-s3-uploader

Flexible and efficient resize, rename, and upload images to Amazon S3 disk storage. Uses the official AWS Node SDK for transfer, and ImageMagick for image processing. Support for multiple image versions targets.
https://www.npmjs.com/package/s3-uploader
MIT License
241 stars 54 forks source link

Please help me on this. #67

Closed kantaharikrishna closed 8 years ago

kantaharikrishna commented 8 years ago

OS: Windows 7, 32-bit Node: 0.12.0 Imagemagick: 6.9.0

client.upload('/uploads/site-logo.png', {}, function(err, versions, meta) {
                  if (err) { throw err; }

                  versions.forEach(function(image) {
                    console.log(image.width, image.height, image.url);
                  });
                });

Please help me on this.

Starefossen commented 8 years ago

What does the error message say?

kantaharikrishna commented 8 years ago
              if (err) { throw err; }
                               ^

ChildProcess.exithandler (child_process.js:744:12),ChildProcess.emit (events.js:110:17),maybeClose (child_process.js:1008:16),Process.ChildProcess._handle.on exit (child_process.js:1080:5)

On Sun, Dec 13, 2015 at 10:58 PM, Hans Kristian Flaatten < notifications@github.com> wrote:

What does the error message say?

— Reply to this email directly or view it on GitHub https://github.com/Turistforeningen/node-s3-uploader/issues/67#issuecomment-164278914 .

Thanks, Harikrishna Kanta

kantaharikrishna commented 8 years ago

Please find the below error:

{ [Error: Command failed: C:\Windows\system32\cmd.exe /s /c "convert uploads/site-logo.png -auto-orient -strip -write mpr:uploads/site-logo.png +delete mpr:u ploads/site-logo.png -quality 80 -resize "1040x1040" -write uploads\site-logo-large.jpg +delete mpr:uploads/site-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "780" -write uploads\site-logo-medium.png +delete mpr:uploads/site-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "320" -write uploads\site-lo go-small.png +delete mpr:uploads/site-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "x100" -write uploads\site-logo-thumb1.png +delete mpr:uploads/sit e-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "250x250" uploads\site-logo-thumb2.png" convert.exe: invalid argument for option `-crop': NaNxNaN+NaN+NaN @ error/convert.c/ConvertImageCommand/1156. ] killed: false, code: 1, signal: null, cmd: 'C:\Windows\system32\cmd.exe /s /c "convert uploads/site-logo.png -auto-orient -strip -write mpr:uploads/site-logo.png +delete mpr:uploads/site-log o.png -quality 80 -resize "1040x1040" -write uploads\site-logo-large.jpg +delete mpr:uploads/site-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "780" -write uploads\site-logo-medium.png +delete mpr:uploads/site-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "320" -write uploads\site-logo-small.png +delete mpr:uploads/site-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "x100" -write uploads\site-logo-thumb1.png +delete mpr:uploads/site-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "250x250" uploads\site-logo-thumb2.png"' } d:\Hari\node\yadva\application\controllers\help_desk.js:229 versions.forEach(function(image) { ^ d:\Hari\node\yadva\application\controllers\help_desk.js:229:27,d:\Hari\node\yadva\node_modules\s3-uploader\lib\index.js:126:12,d:\Hari\node\yadva\node_module s\s3-uploader\node_modules\async\lib\async.js:52:16,d:\Hari\node\yadva\node_modules\s3-uploader\node_modules\async\lib\async.js:543:21,d:\Hari\node\yadva\nod e_modules\s3-uploader\node_modules\async\lib\async.js:173:37,d:\Hari\node\yadva\node_modules\s3-uploader\node_modules\im-resize\index.js:12:21,ChildProcess.e xithandler (child_process.js:751:5),ChildProcess.emit (events.js:110:17),maybeClose (child_process.js:1008:16),Process.ChildProcess._handle.onexit (child_pro cess.js:1080:5)

On Sun, Dec 13, 2015 at 11:02 PM, hari krishna kantaharikrishna@gmail.com wrote:

              if (err) { throw err; }
                               ^

ChildProcess.exithandler (child_process.js:744:12),ChildProcess.emit (events.js:110:17),maybeClose (child_process.js:1008:16),Process.ChildProcess._handle.on exit (child_process.js:1080:5)

On Sun, Dec 13, 2015 at 10:58 PM, Hans Kristian Flaatten < notifications@github.com> wrote:

What does the error message say?

— Reply to this email directly or view it on GitHub https://github.com/Turistforeningen/node-s3-uploader/issues/67#issuecomment-164278914 .

Thanks, Harikrishna Kanta

Thanks, Harikrishna Kanta

kantaharikrishna commented 8 years ago

Issue resolved.

Thank you. On 13 Dec 2015 23:08, "hari krishna" kantaharikrishna@gmail.com wrote:

Please find the below error:

{ [Error: Command failed: C:\Windows\system32\cmd.exe /s /c "convert uploads/site-logo.png -auto-orient -strip -write mpr:uploads/site-logo.png +delete mpr:u ploads/site-logo.png -quality 80 -resize "1040x1040" -write uploads\site-logo-large.jpg +delete mpr:uploads/site-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "780" -write uploads\site-logo-medium.png +delete mpr:uploads/site-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "320" -write uploads\site-lo go-small.png +delete mpr:uploads/site-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "x100" -write uploads\site-logo-thumb1.png +delete mpr:uploads/sit e-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "250x250" uploads\site-logo-thumb2.png" convert.exe: invalid argument for option `-crop': NaNxNaN+NaN+NaN @ error/convert.c/ConvertImageCommand/1156. ] killed: false, code: 1, signal: null, cmd: 'C:\Windows\system32\cmd.exe /s /c "convert uploads/site-logo.png -auto-orient -strip -write mpr:uploads/site-logo.png +delete mpr:uploads/site-log o.png -quality 80 -resize "1040x1040" -write uploads\site-logo-large.jpg +delete mpr:uploads/site-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "780" -write uploads\site-logo-medium.png +delete mpr:uploads/site-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "320" -write uploads\site-logo-small.png +delete mpr:uploads/site-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "x100" -write uploads\site-logo-thumb1.png +delete mpr:uploads/site-logo.png -quality 70 -crop "NaNxNaN+NaN+NaN" -resize "250x250" uploads\site-logo-thumb2.png"' } d:\Hari\node\yadva\application\controllers\help_desk.js:229 versions.forEach(function(image) { ^

d:\Hari\node\yadva\application\controllers\help_desk.js:229:27,d:\Hari\node\yadva\node_modules\s3-uploader\lib\index.js:126:12,d:\Hari\node\yadva\node_module

s\s3-uploader\node_modules\async\lib\async.js:52:16,d:\Hari\node\yadva\node_modules\s3-uploader\node_modules\async\lib\async.js:543:21,d:\Hari\node\yadva\nod

e_modules\s3-uploader\node_modules\async\lib\async.js:173:37,d:\Hari\node\yadva\node_modules\s3-uploader\node_modules\im-resize\index.js:12:21,ChildProcess.e xithandler (child_process.js:751:5),ChildProcess.emit (events.js:110:17),maybeClose (child_process.js:1008:16),Process.ChildProcess._handle.onexit (child_pro cess.js:1080:5)

On Sun, Dec 13, 2015 at 11:02 PM, hari krishna <kantaharikrishna@gmail.com

wrote:

              if (err) { throw err; }
                               ^

ChildProcess.exithandler (child_process.js:744:12),ChildProcess.emit (events.js:110:17),maybeClose (child_process.js:1008:16),Process.ChildProcess._handle.on exit (child_process.js:1080:5)

On Sun, Dec 13, 2015 at 10:58 PM, Hans Kristian Flaatten < notifications@github.com> wrote:

What does the error message say?

— Reply to this email directly or view it on GitHub https://github.com/Turistforeningen/node-s3-uploader/issues/67#issuecomment-164278914 .

Thanks, Harikrishna Kanta

Thanks, Harikrishna Kanta

shravanjha commented 7 years ago

I am having the same issue. How did you solve this?

jlissner commented 7 years ago

@kantaharikrishna would you mind letting myself and @shravanjha know how you resolved the issue? I am having the same issue.

-Thanks. Joe

kantaharikrishna commented 7 years ago

@jlissner @shravanjha options for the Upload class for the updated version has been changed and those changes seems not been updated on document. It's better you contact the author for proper documentation for a required package version.

kkesley commented 7 years ago

hi @kantaharikrishna how do you resolve this issue? I'm having the exact same issue.. What do you change in the upload class if I may ask?

gzurbach commented 7 years ago

In case it helps anyone: I fixed this issue by making sure that the file name passed to client.upload() has an extension. My file was named /tpm/foo/random-name-with-no-enxtension - renaming it with a .jpg at the end fixed the issue.