After an evaluation, GNOME has moved from Bugzilla to GitLab. Learn more about GitLab.
No new issues can be reported in GNOME Bugzilla anymore.
To report an issue in a GNOME project, go to GNOME GitLab.
Do not go to GNOME Gitlab for: Bluefish, Doxygen, GnuCash, GStreamer, java-gnome, LDTP, NetworkManager, Tomboy.
Bug 390153 - Magnatune plugin hangs, charges CC but no music
Magnatune plugin hangs, charges CC but no music
Status: RESOLVED FIXED
Product: rhythmbox
Classification: Other
Component: Plugins (other)
0.9.7
Other Linux
: Normal critical
: ---
Assigned To: RhythmBox Maintainers
RhythmBox Maintainers
Depends on:
Blocks:
 
 
Reported: 2006-12-27 22:00 UTC by David Holmes
Modified: 2007-01-13 05:07 UTC
See Also:
GNOME target: ---
GNOME version: ---


Attachments
fix (1.23 KB, patch)
2006-12-27 23:47 UTC, Adam Zimmerman
committed Details | Review

Description David Holmes 2006-12-27 22:00:31 UTC
OS: Ubuntu 6.10
Rhythmbox 0.9.7 built from source; all dependencies from up-to-date Ubuntu repositories.

I tested the Magnatune plugin twice, with the following results:

First attempt: Purchased "Dead Man's Hand" by "Seismic Anamoly" for $8.  After clicking "Purchase", a window appeared containing just the following text: "Authorizing purchase with Magnatune server. Please wait...".  This window did not disappear and did not make any indication that anything had happened.

I clicked the "X" button, and the window disappeared.  Nothing further happened regarding my purchase, and Rhythmbox continued to function just as it had before I clicked "Purchase Album".

I checked my online banking interface, and discovered that my card had been charged $8.  I never received an email regarding this purchase, telling me where I could download the album.

Second attempt: Purchased "Prospects" by "Chris Juergensen".  This time I was running Rhythmbox from a console.  The exact same thing happened, except this time I did not close the idle "Authorizing..." window (it is still open, an hour later).  This time, I DID receive an email telling me where to download my album, and was able to download it successfully.

The following also appeared on my console; I'm not sure if it's related or not: 

Traceback (most recent call last):
  • File "/home/david/local/lib/rhythmbox/plugins/magnatune/MagnatuneSource.py", line 438 in start_download
    self.__download_album(audio_dl_uri)
  • File "/home/david/local/lib/rhythmbox/plugins/magnatune/MagnatuneSource.py", line 459 in __download_album
    library_location = self.__client.get_list("/apps/rhythmbox/library_locations", gconf.VALUE_STRING)[0] # Just use the first library location
IndexError: list index out of range

Because I'm not certain what the expected behavior is, I cannot delve much further than this.  I am pretty sure that an indefinitely hanging window is not the expected behavior. It could be that I simply typed my email address wrong the first time and thus never received the email, and because I never got to whatever happens after the "Authorizing..." window, I never had any other recourse.

Regardless, I think a little bit of discussion is in order; now that Rhythmbox is causing people's credit cards to be charged, the highest degree of reliability is to be expected, otherwise there'll be a whole world of trouble once this code finds itself in common distribution.

I apologize if this isn't the most informative bug report and is not exactly
reproducible.  If somebody can provide a way to test this system that doesn't charge my credit card, I'd be happy to try and look into it more.  I marked it "blocker" so it would get seen; feel free to change it to "critical" if that's more suitable.
Comment 1 Alex Lancaster 2006-12-27 22:33:44 UTC
Cc'ing the Magnatune plugin author.
Comment 2 David Holmes 2006-12-27 22:37:11 UTC
Okay, the problem was not having a Library path set; after setting one and restarting RB it seems to have automatically started downloading both albums.  So this isn't quite so scary after all.

It's still a pretty substantial bug though, if I may say so.
Comment 3 David Holmes 2006-12-27 22:38:00 UTC
Okay, the problem was not having a Library path set; after setting one and restarting RB it seems to have automatically started downloading both albums.  So this isn't quite so scary after all.

It's still a pretty substantial bug though, if I may say so.
Comment 4 Adam Zimmerman 2006-12-27 23:47:47 UTC
Created attachment 78965 [details] [review]
fix

This will cause a dialog to pop up if there is no library location set when the user clicks the purchase button.

This patch relies on the method name patch from bug 388425 to apply properly. If it isn't, just change purchase_album back to purchase_tracks.
Comment 5 Jonathan Matthew 2007-01-13 05:07:20 UTC
I reworked this a bit to use rb.error_dialog instead of constructing a dialog manually, so the result looks like all other rhythmbox error dialogs.