IPB

Welcome Guest ( Log In | Register )

66 Pages V  « < 64 65 66  
Reply to this topicStart new topic
foo_discogs, Tags files with information from Discogs
SmirG3l
post Feb 16 2015, 15:39
Post #1626





Group: Members
Posts: 6
Joined: 31-August 13
Member No.: 109896



So Suddenly after months of working just fine

QUOTE
(skipped) Error: Authorization Failed (401) [Is OAuth working?] - (url: http://api.discogs.com/oauth/request_token)

[ESCAPE to close]


How do I fix this? it won't let me authenticate it anymore?
Go to the top of the page
+Quote Post
SmirG3l
post Feb 16 2015, 19:07
Post #1627





Group: Members
Posts: 6
Joined: 31-August 13
Member No.: 109896



QUOTE
foo_discogs: HTTP error status: HTTP/1.1 401 Unauthorized
foo_discogs: Exception handling: http://api.discogs.com/oauth/request_token?
Go to the top of the page
+Quote Post
alexinc
post Feb 17 2015, 19:45
Post #1628





Group: Members
Posts: 85
Joined: 12-February 08
Member No.: 51251



i'm quite surprised noone mentiones the series field of discogs yet and after searching the net for further Information on how to tag the Name of a compilation series i didnt find any hint besides "part of an compilation = 1". so there is no "official tag" for storing the Name of an compilation series? if updated, could disogs-plugin write something like %discogs_compilation_series%? smile.gif
Go to the top of the page
+Quote Post
Der Internet
post Feb 21 2015, 00:39
Post #1629





Group: Members
Posts: 57
Joined: 27-December 09
Member No.: 76387



anyone got Problems reaching the discogs-api, too?
Go to the top of the page
+Quote Post
komhst
post Feb 21 2015, 07:06
Post #1630





Group: Members
Posts: 27
Joined: 3-August 10
From: Greece
Member No.: 82789



QUOTE (Der Internet @ Feb 21 2015, 01:39) *
anyone got Problems reaching the discogs-api, too?


Yes, since yesterday. sad.gif
Go to the top of the page
+Quote Post
roscar
post Feb 21 2015, 15:57
Post #1631





Group: Members
Posts: 4
Joined: 29-August 10
Member No.: 83465



I get (wihtout the API)

CODE
(FATAL) Error [release 6533212]: Page Deleted or Missing (404) - (url: http://s.pixogs.com/image/R-6533212-1421560153-6357.jpeg.jpg)


The extraneous extension .jpg is the problem, the correct URL is http://s.pixogs.com/image/R-6533212-1421560153-6357.jpeg. Discogs maybe (just guessing here) changed something to allow for different formats, why listing the full filename including extension makes sense. Either Discogs' server or foo_discogs adds .jpg, because it always did. Should be easily fixable for zoomorph either way.

With the API I get

CODE
(FATAL) Error [release 6533212]: Page Deleted or Missing (404) - (url: http://api-img.discogs.com/###hash###=/fit-in/150x150/filters:strip_icc():format(jpeg):mode_rgb()/discogs-images/R-6533212-1421560153-6357.jpeg.jpg)


... so the API doesn't make any difference.

If you disable art completely (also in preview dialog) everything else works as normal.

Edit: I wrote an issue on the bugtracker.

This post has been edited by roscar: Feb 21 2015, 16:05
Go to the top of the page
+Quote Post
komhst
post Feb 21 2015, 16:21
Post #1632





Group: Members
Posts: 27
Joined: 3-August 10
From: Greece
Member No.: 82789



QUOTE (roscar @ Feb 21 2015, 16:57) *
I get (wihtout the API)

CODE
(FATAL) Error [release 6533212]: Page Deleted or Missing (404) - (url: http://s.pixogs.com/image/R-6533212-1421560153-6357.jpeg.jpg)


The extraneous extension .jpg is the problem, the correct URL is http://s.pixogs.com/image/R-6533212-1421560153-6357.jpeg. Discogs maybe (just guessing here) changed something to allow for different formats, why listing the full filename including extension makes sense. Either Discogs' server or foo_discogs adds .jpg, because it always did. Should be easily fixable for zoomorph either way.


All the cover images in discogs have filenames such as this: R-3869373-1347520948-2326.jpeg.jpg

This post has been edited by komhst: Feb 21 2015, 16:23
Go to the top of the page
+Quote Post
roscar
post Feb 21 2015, 18:29
Post #1633





Group: Members
Posts: 4
Joined: 29-August 10
Member No.: 83465



QUOTE (komhst @ Feb 21 2015, 17:21) *
All the cover images in discogs have filenames such as this: R-3869373-1347520948-2326.jpeg.jpg


No, they don't - as I've already shown. But that's the filename you get through their crappy lightbox viewer.
Go to the top of the page
+Quote Post
zoomorph
post Feb 21 2015, 19:27
Post #1634





Group: Developer
Posts: 87
Joined: 16-August 14
Member No.: 117002



It looks like Discogs changed something with their images API causing existing applications to break. rolleyes.gif

I will look into this tomorrow. In the meantime, I guess users need to wait, or disable downloading images.
Go to the top of the page
+Quote Post
zoomorph
post Feb 21 2015, 20:45
Post #1635





Group: Developer
Posts: 87
Joined: 16-August 14
Member No.: 117002



OK, I've uploaded a new version (1.52) which should fix download images.

The problem was that foo_discogs was still using OAuth for images, but the new images endpoint doesn't support OAuth. Instead, it now has authentication built directly into the URL. Sadly, this means that it's no longer trivial to bypass the API for images. My fingers are crossed that the new built-in authentication for images is on a per-IP basis rather than a per application.... If not, we'll probably be hitting the 1000-images per day limit again soon. Maybe if every foo_discogs user goes and complains on the Discogs API Forum they'll fix that limit to be per-IP instead of per-application. ;-)

This post has been edited by zoomorph: Feb 21 2015, 20:47
Go to the top of the page
+Quote Post
komhst
post Feb 22 2015, 08:48
Post #1636





Group: Members
Posts: 27
Joined: 3-August 10
From: Greece
Member No.: 82789



It's working perfectly now!
Thanks!
Go to the top of the page
+Quote Post
alexinc
post Feb 23 2015, 18:51
Post #1637





Group: Members
Posts: 85
Joined: 12-February 08
Member No.: 51251



a big thx from me too! smile.gif

i noticed one little bug, when having an artist name consisting of more than 2 artists combined with "features" or "presents", foo_discogs always fetches the first one and uses it for the following ones. so "abc pres. def feat. xyz" always becomes "abc feat. def feat. xyz" e.g. smile.gif

This post has been edited by alexinc: Feb 23 2015, 18:54
Go to the top of the page
+Quote Post
zoomorph
post Feb 23 2015, 21:58
Post #1638





Group: Developer
Posts: 87
Joined: 16-August 14
Member No.: 117002



It sounds like the crappy image rate-limiting has been fixed (it's now done per-IP and allowed 300/min, unofficial). That's good news as it means foo_discogs users shouldn't have any more problems.
Go to the top of the page
+Quote Post
zoomorph
post Feb 25 2015, 07:56
Post #1639





Group: Developer
Posts: 87
Joined: 16-August 14
Member No.: 117002



QUOTE (alexinc @ Feb 23 2015, 18:51) *
a big thx from me too! smile.gif

i noticed one little bug, when having an artist name consisting of more than 2 artists combined with "features" or "presents", foo_discogs always fetches the first one and uses it for the following ones. so "abc pres. def feat. xyz" always becomes "abc feat. def feat. xyz" e.g. smile.gif

Please provide example release.
Go to the top of the page
+Quote Post
alexinc
post Feb 25 2015, 16:12
Post #1640





Group: Members
Posts: 85
Joined: 12-February 08
Member No.: 51251



QUOTE (zoomorph @ Feb 25 2015, 07:56) *
Please provide example release.


http://www.discogs.com/release/654021 Track 2-10
http://www.discogs.com/release/1229612 Track 1-4
http://www.discogs.com/release/1485101 Track 1.17 smile.gif

was tagging quite some compilations the last days and noticed that behaviour on various releases.

This post has been edited by alexinc: Feb 25 2015, 16:15
Go to the top of the page
+Quote Post
zoomorph
post Feb 26 2015, 20:15
Post #1641





Group: Developer
Posts: 87
Joined: 16-August 14
Member No.: 117002



Thanks. Fixed that bug for the next version.
Go to the top of the page
+Quote Post
alexinc
post Feb 27 2015, 11:42
Post #1642





Group: Members
Posts: 85
Joined: 12-February 08
Member No.: 51251



that was fast, thx! could you plz consider adding fetching the series-field in upcoming version? belonging links posted previously belong to series "trancemaster" e.g. smile.gif
Go to the top of the page
+Quote Post

66 Pages V  « < 64 65 66
Reply to this topicStart new topic
1 User(s) are reading this topic (1 Guests and 0 Anonymous Users)
0 Members:

 



RSS Lo-Fi Version Time is now: 2nd March 2015 - 16:01