-
Notifications
You must be signed in to change notification settings - Fork 13
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
getting 'HTTP Error 400: Bad Request' on submission #57
Comments
Hm, non-ascii passwords, that can be a problem in my script, in musicbrainzngs, in the musicbrainz server (web service part) or even at another place (python?) What isrcsubmit version and/or package (isrcsubmit-*-mac.zip) are you using? I couldn't reproduce your "400: Bad Request", yet, but I get a |
The test script is available at https://gist.github.com/JonnyJD/5429281. Please post the output of that script and |
In case this isn't related (as in you also don't get the 400 error using my password_test.py) I need the output of isrcsubmit.py when this 400 error occurs. Inluding the disc ID and the ISRCs. FYI: |
If possible, running isrcsubmit (current master) with |
I opened a new ticket for the "401: Digest Auth failed" exception: #58 . There is a ticket to expose the exact 400 error in alastair/python-musicbrainzngs#80. |
The password is all ascii. I use 1password to randomly generate passwords, which includes ascii symbols, since it's a web service I imagined would need URL escaping - I could be wrong about that, I don't know how the MB API works. I'll run the tests you asked for now. |
Python 2.7.2 python packages installed with pip: libdiscid installed with brew: discisrc from the isrcsubmit-1.0.1-mac package it doesn't seem to matter what I put in usename and password: ./password_test.py Please input your MusicBrainz username: [my username] ERROR: Leaving authenticate OR just entering user: test123 password: 123 /password_test.py Please input your MusicBrainz username: test123 ERROR: Leaving authenticate |
and here's the most recent -dev version: python isrcsubmit.py -d 1 --debug using discisrc DiscID: kmtZWQ.s_5hxRGCGT4gxzK9MhfU- Artist: Various Artists Warning: Unable to read the disc's media catalog number. Do you want to submit? [y/N] y Please input your MusicBrainz username: [my username] ERROR: Couldn't send ISRCs: caused by: HTTP Error 400: Bad Request |
Thanks a lot for your tests. I made a couple semi-related changes to the code based on your test data. However, I could reproduce your problem. The problem you seem to have is probably http://bugs.python.org/issue4140 When you can get around using a proxy things will work with current master (I added changes after your tests), but if you don't and things need to be fixed on MB server then this takes at least 2 weeks (due to the release cycle for MB server). EDIT: |
Reported the proxy problem to MusicBrainz-server with http://tickets.musicbrainz.org/browse/MBS-6185. EDIT: |
FYI: |
That did the trick - the proxy I was behind is just for ad-blocking purposes. I turned that off and it went through no problem. An obnoxious bug, but at least I managed to shake out a bunch of lower-hanging fruit as well. |
Is this still an issue? |
This is possibly a problem with musicbrainzngs. Possible cause is that my password uses a few unusual symbols - one or more of these may not be being escaped properly. It's not throwing an authentication error, however, which is what I'd expect looking at the code. On the other hand, I don't get an authentication error even when entering bogus credentials, that generates a 400: Bad Request as well. So I'm not sure what's going on there.
EDIT:
The problem for this 400 error is a proxy.
Another sources of 400 errors could be #67.
This is blocked on https://rt.cpan.org/Public/Bug/Display.html?id=84789 (upstream of MB server)
The text was updated successfully, but these errors were encountered: