feedbin / feedbin-api

Feedbin API Documentation
349 stars 26 forks source link

extracted_content_url Missing From Extended Mode #31

Closed robmathers closed 5 years ago

robmathers commented 5 years ago

It appears that the extracted_content_url parameter is missing from the entry payload when extended mode is enabled.

Example (auth removed):

 curl "https://api.feedbin.com/v2/entries/2176629936.json"

{
   "summary" : "Baltimore has allowed an AL-record 245 homers (56 by the Yankees), 13 from the season mark set by Cincinnati in 2016.",
   "title" : "Yankees put on power display against Orioles",
   "id" : 2176629936,
   "url" : "https://www.thestar.com/sports/baseball/2019/08/12/yankees-put-on-power-display-against-orioles.html",
   "extracted_content_url" : "https://extract.feedbin.com/parser/feedbin/384a1f22aad4f6aa933f2812dfa2cbe139a9c44a?base64_url=aHR0cHM6Ly93d3cudGhlc3Rhci5jb20vc3BvcnRzL2Jhc2ViYWxsLzIwMTkvMDgvMTIveWFua2Vlcy1wdXQtb24tcG93ZXItZGlzcGxheS1hZ2FpbnN0LW9yaW9sZXMuaHRtbA==",
   "feed_id" : 1023350,
   "created_at" : "2019-08-13T02:50:05.380370Z",
   "content" : "<img src=\"https://www.thestar.com/content/dam/thestar/sports/baseball/2019/08/12/yankees-put-on-power-display-against-orioles/yankees_slide.jpg\"><br>Baltimore has allowed an AL-record 245 homers (56 by the Yankees), 13 from the season mark set by Cincinnati in 2016.",
   "published" : "2019-08-13T02:47:07.000000Z",
   "author" : "Ronald Blum - The Associated Press"
}
curl "https://api.feedbin.com/v2/entries/2176629936.json?mode=extended"

{
   "feed_id" : 1023350,
   "created_at" : "2019-08-13T02:50:05.380370Z",
   "original" : null,
   "id" : 2176629936,
   "enclosure" : null,
   "content" : "<img src=\"https://www.thestar.com/content/dam/thestar/sports/baseball/2019/08/12/yankees-put-on-power-display-against-orioles/yankees_slide.jpg\"><br>Baltimore has allowed an AL-record 245 homers (56 by the Yankees), 13 from the season mark set by Cincinnati in 2016.",
   "title" : "Yankees put on power display against Orioles",
   "extracted_articles" : [],
   "summary" : "Baltimore has allowed an AL-record 245 homers (56 by the Yankees), 13 from the season mark set by Cincinnati in 2016.",
   "twitter_id" : null,
   "images" : {
      "original_url" : "https://www.thestar.com/content/dam/thestar/sports/baseball/2019/08/12/yankees-put-on-power-display-against-orioles/yankees_slide.jpg",
      "size_1" : {
         "height" : 304,
         "cdn_url" : "https://images.feedbinusercontent.com/66d5809/66d5809ba5b857ea441863396b6d7bea46403d01.jpg",
         "width" : 542
      }
   },
   "published" : "2019-08-13T02:47:07.000000Z",
   "twitter_thread_ids" : [],
   "json_feed" : null,
   "author" : "Ronald Blum - The Associated Press",
   "url" : "https://www.thestar.com/sports/baseball/2019/08/12/yankees-put-on-power-display-against-orioles.html"
}

I see the same behaviour whether I use /entries.json, /entries.json?ids=1234 or /entries/1234.json.

The sample in the About Extended Mode docs suggests that extended mode should have extracted_content_url when available. Is that erroneous, or can it be added to extended mode (I hope so)?

Thanks

benubois commented 5 years ago

Good catch! Should be fixed now.

robmathers commented 5 years ago

Working for me, thanks for the quick fix!