Fix #214: Do not crash on flac import if musicbrainz tags are missing

api/funkwhale_api/music/metadata.py

@@ -30,7 +30,7 @@ def get_id3_tag(f, k):
 
 def get_flac_tag(f, k):
     try:
-        return f.get(k)[0]
+        return f.get(k, [])[0]
     except (KeyError, IndexError):
         raise TagNotFound(k)
 
@@ -158,6 +158,9 @@ CONF = {
             'musicbrainz_recordingid': {
                 'field': 'musicbrainz_trackid'
             },
+            'test': {
+                'field': 'test'
+            },
         }
     },
 }

api/tests/music/test_metadata.py

@@ -57,3 +57,11 @@ def test_can_get_metadata_from_flac_file(field, value):
     data = metadata.Metadata(path)
 
     assert data.get(field) == value
+
+
+def test_can_get_metadata_from_flac_file_not_crash_if_empty():
+    path = os.path.join(DATA_DIR, 'sample.flac')
+    data = metadata.Metadata(path)
+
+    with pytest.raises(metadata.TagNotFound):
+        data.get('test')

changes/changelog.d/214.bugfix

@@ -0,0 +1 @@
+Do not crash on flac import if musicbrainz tags are missing (#214)