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

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

    XMLWordPrintable

    Details

    • Type: Bug
    • Status: Closed
    • Priority: Normal
    • Resolution: Fixed
    • Affects Version/s: None
    • Fix Version/s: 2019-07-01.0
    • Component/s: None
    • Labels:

      Description

      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'
      

        Attachments

          Activity

            People

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

              Dates

              • Created:
                Updated:
                Resolved:

                Packages

                Version Package
                2019-07-01.0