Allow for cddb response 500
authorAndrew Strong <andrew.david.strong@gmail.com>
Thu, 17 Dec 2015 01:32:19 +0000 (12:32 +1100)
committerAndrew Strong <andrew.david.strong@gmail.com>
Thu, 17 Dec 2015 01:32:19 +0000 (12:32 +1100)
Thanks again to Von Welch for the bug report and patch.
This closes Issue 26:

http://abcde.einval.com/bugzilla/show_bug.cgi?id=26

abcde
changelog

diff --git a/abcde b/abcde
index 807ea92..732bdcd 100755 (executable)
--- a/abcde
+++ b/abcde
@@ -2669,12 +2669,14 @@ do_cddbread ()
                        done
                        echo >> "$ABCDETEMPDIR/cddbchoices"
                        ;;
-               202|403|409|503)
-                       # No match
+               202|403|409|500|503)
+                       # TODO: Explain these error codes a little more accurately:
+                       # http://ftp.freedb.org/pub/freedb/misc/freedb_CDDB_protcoldoc.zip
+                       # No match response:
                        case "$RESPONSECODE" in
                        202) echo "No CDDB match." >> "$ABCDETEMPDIR/cddbchoices" ;;
                        403|409) echo "CDDB entry is corrupt, or the handshake failed." >> "$ABCDETEMPDIR/cddbchoices" ;;
-                       503) echo "CDDB unavailable." >> "$ABCDETEMPDIR/cddbchoices" ;;
+                       500|503) echo "CDDB unavailable." >> "$ABCDETEMPDIR/cddbchoices" ;;
                        esac
                        $CDDBTOOL template $(cat "$ABCDETEMPDIR/discid") > "$ABCDETEMPDIR/cddbread.0"
                        # List out disc title/author and contents of template
index a16a181..c6f2187 100644 (file)
--- a/changelog
+++ b/changelog
@@ -1,5 +1,8 @@
 abcde 2.7.2
 
+  * Allow for cddb response 500. Thanks again to Von Welch for the
+    bug report and patch. This closes Issue 26:
+    http://abcde.einval.com/bugzilla/show_bug.cgi?id=26
   * Fix for 'expansion of $REDIR' bug on MacOSX. Thanks to Von Welch
     for the bug report. This closes Issue 22:
     http://abcde.einval.com/bugzilla/show_bug.cgi?id=22