Support for Drupal 7 is ending on 5 January 2025—it’s time to migrate to Drupal 10! Learn about the many benefits of Drupal 10 and find migration tools in our resource center.
According to the REST CCU docs the SOAP API has been deprecated. I don't know when the SOAP API will be sunsetted, but is anyone working on updating this module to support the REST API?
Reference: https://api.ccu.akamai.com/ccu/v2/docs/index.html (search the page for "deprecated" for the reference to the SOAP API)
Comment | File | Size | Author |
---|---|---|---|
#22 | akamai-2134145-use-rest-api-22.patch | 6.99 KB | Barrett |
#11 | akamai-2134145-use-rest-api-11.patch | 6.19 KB | joelcollinsdc |
#9 | akamai-2134145-use-rest-api-8.patch | 6.34 KB | joelcollinsdc |
#8 | akamai-restapi-2134145-8.patch | 8.41 KB | cherrypj |
#5 | akamai-restapi-000-D7.patch | 7.05 KB | roshan_shetty |
Comments
Comment #1
cherrypj CreditAttribution: cherrypj commentedA new and improved REST API is now available to purge content off the Platform.
Existing users must migrate to the new API by June 1, 2014.
If you are currently using the SOAP based API for purge, or if you intend to start using API to purge your content, please read on.
The benefits of the new API are:
What does it mean for me?
Comment #2
Barrett CreditAttribution: Barrett commentedBumping the priority on this, since after June the module will be non-functional without this issue being resolved.
Comment #3
joelcollinsdc CreditAttribution: joelcollinsdc commentedComment #4
joelcollinsdc CreditAttribution: joelcollinsdc commentedComment #5
roshan_shetty CreditAttribution: roshan_shetty commentedComment #6
Barrett CreditAttribution: Barrett commentedThere are a few issues with the patch from comment 5, one conceptual (the first one below) and a couple related to execution of the patch.
Lines removed because of basing on 1.3
Comment #7
Barrett CreditAttribution: Barrett commentedThe above not withstanding, thank you roshan_shetty for taking a first cut at this. It's very much appreciated.
Comment #8
cherrypj CreditAttribution: cherrypj commentedI've taken roshan_shetty's work and made some improvements, including those suggested by Barrett. This work also pulls in issue # 2183829.
I've tested locally, against my Akamai account, but I welcome improvements, as well as suggestions for making this more drupal-y.
Comment #9
joelcollinsdc CreditAttribution: joelcollinsdc commentedDamn. CherryPi beat me to it.
Cleaned up patch and made some updates.
I've been workign here if someone wants to use git instead of living in patchland
http://git.drupal.org/sandbox/joelcollinsdc/2237671.git
Comment #10
joelcollinsdc CreditAttribution: joelcollinsdc commentedI dont see a reason we need to use curl for this instead of drupal_http_request, I think that is the next thing to work on.
Comment #11
joelcollinsdc CreditAttribution: joelcollinsdc commentednow using drupal_http_request
Comment #12
cherrypj CreditAttribution: cherrypj commentedGreat stuff, joelcollinsdc--that was my next step. One thing I did note is that cURL didn't work since the REST URL is https, so I had to add in:
(I should use TRUE for verifypeer, I think.)
Does
drupal_http_request
take that into account?Also, there's no need to leave the SOAP stuff in there.
Comment #13
Barrett CreditAttribution: Barrett commentedThe patch in comment 11 looks good, though I do get a complaint about a blank line at EOP in the .info file
That said, I'm currently without an account at Akamai I can test against. Has it been verified to actually work, including handling the settings for Action Type and Domain?
I'm also rethinking my initial opinion about ripping out the SOAP options. If we leave them in for now, a) we'll be able to get this out faster by skipping the work necessary to remove them, and b) those using SOAP will be able to continue using until Akamai actually sunsets the service on their. After June 1, we should definitely remove it, but between now and then I'm thinking it should stay. What do y'all think?
Comment #14
azinck CreditAttribution: azinck commentedShould definitely leave the SOAP stuff in there so that users can update the module without breaking a working configuration.
Comment #15
joelcollinsdc CreditAttribution: joelcollinsdc commentedI've tested with our Akamai API., yes
Comment #16
Barrett CreditAttribution: Barrett commentedAwesome. @cherrypj, @azinck, can one of you (or anyone else) confirm that @joelcollinsdc's patch in comment 11 works as expected? If we can get independent confirmation, I think we can call this thing RTBC and hopefully get it committed.
Comment #17
joelcollinsdc CreditAttribution: joelcollinsdc commentedOne thing that I didnt test (that I should have) is that the SOAP stuff wasn't adversely affected by adding in the REST api stuff. We shoudl make sure that both apis can be used interchangably.
Comment #18
cherrypj CreditAttribution: cherrypj commentedI can confirm that this module works for both SOAP and REST (if I check the Rest API default checkbox).
Comment #19
roshan_shetty CreditAttribution: roshan_shetty commented#11 fixes work well for me. Only thing I would suggest is adding $response_data->progressUri (Also "submissionTime" if you think it makes sense.) to the watchdog since that is the only way to track back the urls. (no email confirmation with Soap ccu.)
Thanks
Comment #20
joelcollinsdc CreditAttribution: joelcollinsdc commentedHmm... The progressUri is only useful to computers though, right? Its not like you can click on it... It would be cool to have another page that listed the queue of purges, but then you have to keep track of the purgeIds somehow.
Here are the options:
estimatedSeconds
progressUri
purgeId
supportId
httpStatus
detail
pingAfterSeconds
Comment #21
joelcollinsdc CreditAttribution: joelcollinsdc commentedwe probably should put in a hook_requirements in there to alert people that they need to switch to REST if they have the SOAP url still configured. There is no way to automatically switch over from SOAP to rest via update hook, right? Or are the URLs always the same for everyone?
Comment #22
Barrett CreditAttribution: Barrett commentedThe attached patch adds a REQUIREMENT_WARNING message if the akamai_rest_default variable is 0.
I like the progress idea @roshan_shetty raised, but it seems like it should be a separate feature request.
Comment #23
joelcollinsdc CreditAttribution: joelcollinsdc commentedLooks good to me! RTBC?
Comment #24
Barrett CreditAttribution: Barrett commentedW00t! Thanks for all your work on this folks!
Comment #26
Barrett CreditAttribution: Barrett commentedComment #32
cherrypj CreditAttribution: cherrypj commentedAs near as I can tell, there's no checking if the module cannot reach Akamai when the REST API is enabled.
For the SOAP call, I see this:
This exception is helpful if, for whatever reason, the Akamai service can't be reached. (Say a firewall rule or whatever; Akamai adds new IP address *all* the time.)
I see nothing of the sort for the REST API. Am I missing something?