I received an email informing me that there will be an API change and that the URLs should be changed:

Currently, it is possible to make requests using the base URLs http://sendgrid.com/api/ or https://sendgrid.com/api/. We will be deprecating these URLs on February 9th, 2016, and all SendGrid API calls must use https://api.sendgrid.com from that date on.

I checked sendgrid_integration.module and on line 88 it is:
$server = $protocol . 'sendgrid.com/api/mail.send.json';

Is it ok if I change that line to:
$server = $protocol . 'api.sendgrid.com/mail.send.json';
?

Thanks and sorry for not committing a patch (I don't know how to)

Comments

federico created an issue. See original summary.

exlin’s picture

exlin CreditAttribution: exlin at Wunder commented

As long API doesn't change it should be ok. Functionality after changing URL should be relatively easy to test.

  • Perignon committed c82ac24 on 6.x-1.x 
    Issue #2612550: D6 Version: API update. Documentation
Perignon’s picture

Perignon CreditAttribution: Perignon commented
Status: Active » Needs review

Just pushed an update. Let me know if it works. I do not have Drupal 6 installed to do any testing. Actually have never touched D6... :-D

Perignon’s picture

Perignon CreditAttribution: Perignon commented
Status: Needs review » Closed (fixed)
enekoalonso’s picture

enekoalonso CreditAttribution: enekoalonso as a volunteer commented

Hello guys!

The actual URL, as seen at the www.sendgrid.com home page is api.sendgrid.com/api/mail.send.json

The code should read:
$server = $protocol . 'api.sendgrid.com/api/mail.send.json';

Note the two mentions of "api" on subdomain and url path.

Eneko

enekoalonso’s picture

enekoalonso CreditAttribution: enekoalonso as a volunteer commented
Status: Closed (fixed) » Active

See my previous comment ^^

enekoalonso’s picture

enekoalonso CreditAttribution: enekoalonso as a volunteer commented

@Perignon, I'd be happy to contribute the code change, never done it before. I have a D6 site running with this module and the URL I pasted works well.

Regards,
Eneko

  • Perignon committed cefae25 on 6.x-1.x 
    Issue #2612550 by enekoalonso, Perignon: D6 Version: API update
Perignon’s picture

Perignon CreditAttribution: Perignon commented

Update is going into 6.x-1.x-dev. Give it a spin and let me know if that fixes the problem.

enekoalonso’s picture

enekoalonso CreditAttribution: enekoalonso as a volunteer commented
Status: Active » Closed (fixed)

The URL update works great, thanks! Closing the issue as fixed.

Perignon’s picture

Perignon CreditAttribution: Perignon commented

Woohoo!