Bug 757425

Summary: [abrt] picard-0.16-1.fc16: webservice.py:76:__getattr__:AttributeError: response
Product: [Fedora] Fedora Reporter: Jeffrey C. Ollie <jeff>
Component: picardAssignee: Alex Lancaster <alex>
Status: CLOSED ERRATA QA Contact: Fedora Extras Quality Assurance <extras-qa>
Severity: unspecified Docs Contact:
Priority: unspecified    
Version: 16CC: alex, rudd-o
Target Milestone: ---   
Target Release: ---   
Hardware: x86_64   
OS: Unspecified   
Whiteboard: abrt_hash:a5a8302374c92c0682ba83daeeacbb7ae2eacd57
Fixed In Version: Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of: Environment:
Last Closed: 2012-06-20 19:28:52 UTC Type: ---
Regression: --- Mount Type: ---
Documentation: --- CRM:
Verified Versions: Category: ---
oVirt Team: --- RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: --- Target Upstream Version:

Description Jeffrey C. Ollie 2011-11-27 00:16:46 UTC
libreport version: 2.0.7
abrt_version:   2.0.6
cmdline:        /usr/bin/python /usr/bin/picard
comment:        Scanning files using chromaprint
executable:     /usr/bin/picard
kernel:         3.1.0-7.fc16.x86_64
reason:         webservice.py:76:__getattr__:AttributeError: response
time:           Sat 26 Nov 2011 06:15:54 PM CST
uid:            500
username:       jcollie

backtrace:
:webservice.py:76:__getattr__:AttributeError: response
:
:Traceback (most recent call last):
:  File "/usr/lib64/python2.7/site-packages/picard/webservice.py", line 206, in _process_reply
:    handler(xml_handler.document, reply, error)
:  File "/usr/lib64/python2.7/site-packages/picard/acoustid.py", line 59, in _on_lookup_finished
:    for result in document.response[0].results[0].children.get('result', []):
:  File "/usr/lib64/python2.7/site-packages/picard/webservice.py", line 76, in __getattr__
:    raise AttributeError, name
:AttributeError: response
:
:Local variables in innermost frame:
:self: {'attribs': {}, 'text': u'', 'children': {}}
:name: 'response'

Comment 1 Rudd-O DragonFear 2012-02-21 19:36:55 UTC
scanning a file with acoustid fingerprinting

Package: picard-0.16-1.fc16
OS Release: Fedora release 16 (Verne)

Comment 2 Rudd-O DragonFear 2012-02-21 19:47:48 UTC
self.tagger.xmlws.query_acoustid(partial(self._on_lookup_finished, next, file), **params) in acoustid.py calls _on_lookup_finished with a weird document object that does not have a response attribute.

xmlws is a XmlWebService() assigned in the Tagger class in tagger.py

XmlWebService comes from picard's webservice.py and is a Qt object.  query_acoustid there creates a body and self.post() that body.  they implement an HTTP client that does post in that class (Bad form, as python already has one).  the post gets added to self.active_requests.

When a reply comes back, self._process_reply pops the requests from active_Requests  and then it creates an XmlHandler with a .document, which then gets dispatched to the handler (_on_lookup_finished partial call).

This .document has no response attribute.

And that's why Picard barfs.

End of story.

Comment 3 Rudd-O DragonFear 2012-02-21 19:48:07 UTC
(That was the missing traceback from async style programming code.  It sucks because of that.)

Comment 4 Alex Lancaster 2012-02-21 21:33:28 UTC
Thanks for the explanation. As I haven't been able to run into this error myself,  can you check if this already posted upstream: http://tickets.musicbrainz.org/browse/PICARD   If you can't let me know, I'll try to post a bug.

Comment 5 Rudd-O DragonFear 2012-02-22 00:55:30 UTC
Sorry, I have no fucking idea how to post a bug in that crap Web site.  :-)

Comment 6 Fedora Update System 2012-06-04 19:23:56 UTC
picard-1.0-1.fc17 has been submitted as an update for Fedora 17.
https://admin.fedoraproject.org/updates/picard-1.0-1.fc17

Comment 7 Fedora Update System 2012-06-04 19:53:01 UTC
picard-1.0-1.fc16 has been submitted as an update for Fedora 16.
https://admin.fedoraproject.org/updates/picard-1.0-1.fc16

Comment 8 Fedora Update System 2012-06-07 02:38:10 UTC
Package picard-1.0-1.fc17:
* should fix your issue,
* was pushed to the Fedora 17 testing repository,
* should be available at your local mirror within two days.
Update it with:
# su -c 'yum update --enablerepo=updates-testing picard-1.0-1.fc17'
as soon as you are able to.
Please go to the following url:
https://admin.fedoraproject.org/updates/FEDORA-2012-8916/picard-1.0-1.fc17
then log in and leave karma (feedback).

Comment 9 Fedora Update System 2012-06-20 19:28:52 UTC
picard-1.0-1.fc17 has been pushed to the Fedora 17 stable repository.  If problems still persist, please make note of it in this bug report.

Comment 10 Fedora Update System 2012-06-20 19:30:25 UTC
picard-1.0-1.fc16 has been pushed to the Fedora 16 stable repository.  If problems still persist, please make note of it in this bug report.