search

aheckmann / node-ses

An Amazon SES api for nodejs with proper error handling.
http://aheckmann.github.com/node-ses
MIT License
200 stars 37 forks source link

Invalid protocol error #9

Closed craigmurraydev closed 9 years ago

craigmurraydev commented 10 years ago

This is probably something on my end, but I'm not really sure.

/var/app/current/node_modules/aws-sdk/lib/sequential_executor.js:228 throw err; ^ Error: Invalid protocol at Request.init (/var/app/current/node_modules/node-ses/node_modules/request/main.js:274:31) at new Request (/var/app/current/node_modules/node-ses/node_modules/request/main.js:102:8) at request (/var/app/current/node_modules/node-ses/node_modules/request/main.js:800:11) at post (/var/app/current/node_modules/node-ses/lib/ses.js:275:3) at Email.send (/var/app/current/node_modules/node-ses/lib/ses.js:241:3) at SESClient.sendemail (/var/app/current/node_modules/node-ses/lib/ses.js:59:9)

aheckmann commented 10 years ago

This isn't maintained anymore. Do you want commit access to make the fixes?— Sent from Mailbox

On Thu, Aug 14, 2014 at 3:45 PM, craigmurraydev notifications@github.com wrote:

This is probably something on my end, but I'm not really sure. /var/app/current/node_modules/aws-sdk/lib/sequential_executor.js:228 throw err; ^ Error: Invalid protocol at Request.init (/var/app/current/node_modules/node-ses/node_modules/request/main.js:274:31) at new Request (/var/app/current/node_modules/node-ses/node_modules/request/main.js:102:8) at request (/var/app/current/node_modules/node-ses/node_modules/request/main.js:800:11) at post (/var/app/current/node_modules/node-ses/lib/ses.js:275:3) at Email.send (/var/app/current/node_modules/node-ses/lib/ses.js:241:3)

at SESClient.sendemail (/var/app/current/node_modules/node-ses/lib/ses.js:59:9)

Reply to this email directly or view it on GitHub: https://github.com/aheckmann/node-ses/issues/9

markstos commented 9 years ago

I'm not the bug poster above, but I'm interested in commit access for maintenance. You can check my public Github profile to see my contributions to other open open source projects.

One revision I'm interested is to make a new version that that's not complete backcompat that goes ahead and parses the XML error responses back to JSON. That seems more natural for a node API.

Unless there's another module that's recommended instead for sending email through SES via node. I like the simplicity of this one.

To @craigmurraydev: To get your bug fixed you'll need to provide an example case that triggers the above error.

aheckmann commented 9 years ago

@markstos added you. thanks!

markstos commented 9 years ago

I'm closing this issue for now, as no specific fix is being proposed, and it's not entirely clear what the issue is. If someone provides a patch or can point to a specific place in node-ses where there's a bug, let's re-open this or create a new ticket.