• Resolved Cristina

    (@darthcena)


    I’m using this plugin on a client’s site, and she got the following email from GoodReads:

    You’re receiving this email because you have a Goodreads API Developer account which has made plain HTTP requests in the past two weeks.
    This is a final reminder that we will be enabling HTTP to HTTPS redirection for all requests to goodreads.com next week on Monday, December 4th.
    To prevent your applications from breaking, please do one of the following before December 4th:
    Make sure your applications support HTTPS redirects
    Update your applications to make only HTTPS requests
    If you have any questions, please post in the developer forum.
    Best,
    Goodreads

    Anything we need to worry about?

Viewing 3 replies - 1 through 3 (of 3 total)
  • Plugin Author jhanbackjr

    (@jhanbackjr)

    Hi there.

    GoodReviews determines whether to make HTTP or HTTPS requests based on the site on which it is installed. I don’t think you’ll see any problems as a result of the change. In an upcoming update to GoodReviews, I will modify it so that it makes only HTTPS requests.

    Thanks for using GoodReviews and let me know if I can be of further assistance.

    James

    Thread Starter Cristina

    (@darthcena)

    Wonderful, thanks James!

    Plugin Author jhanbackjr

    (@jhanbackjr)

    No problem at all. If you want to test HTTPS-only in the meantime, you can replace line 1205 in jhgrclasses.php. The original line is:

    $jhgrURL = $this->jhgrIsSSL() . 'www.goodreads.com' . $this->jhgrGetIDType($jhgrSCAtts["isbn"],$jhgrSCAtts["grid"]);

    Modify it so that it is instead:

    $jhgrURL = 'https://www.goodreads.com' . $this->jhgrGetIDType($jhgrSCAtts["isbn"],$jhgrSCAtts["grid"]);

    Because the Goodreads API will support only HTTPS after Dec. 4, I’ll be removing the jhgrIsSSL() function completely and using the new line above to form the URL that makes the API request.

    Thanks again for using GoodReviews.

Viewing 3 replies - 1 through 3 (of 3 total)
  • The topic ‘HTTP to HTTPS Request on GoodReads Dev API’ is closed to new replies.