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

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

    • 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'
      

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

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

            • 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

                          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