Uploaded image for project: 'Zapped: AcousticBrainz'
  1. Zapped: AcousticBrainz
  2. AB-411

Can't access items when uuid has upper-case characters

XMLWordPrintable

    • Icon: Bug Bug
    • Resolution: Fixed
    • Icon: Normal Normal
    • 2019-07-01.0
    • None
    • None

      This specific API error happened as a result of changes we made in AB-406, but there is also an issue where flask's <uuid> type in a route doesn't accept upper-case characters unless you pass strict=False into the type definition: https://github.com/wbolster/flask-uuid

      We should accept lower-case, mixed, and upper-case uuids. In the case of mixed and upper, should we redirect to lower-case?

      https://sentry.metabrainz.org/metabrainz/acousticbrainz/issues/89235/

      KeyError: 'D57d7065-020f-4648-B1ca-12c9ba72f78d'
(3 additional frame(s) were not displayed)
...
  File "webserver/decorators.py", line 44, in wrapped_function
    resp = make_response(f(*args, **kwargs))
  File "webserver/views/api/legacy.py", line 33, in get_low_level
    return jsonify(db.data.load_low_level(mbid, offset))

KeyError: 'D57d7065-020f-4648-B1ca-12c9ba72f78d'

            Unassigned Unassigned
            alastairp Alastair Porter
            Votes:
            0 Vote for this issue
            Watchers:
            2 Start watching this issue

              Created:
              Updated:
              Resolved:

                Version Package
                2019-07-01.0