code-review@v1: add code-review comment:

✨ PR Review

The PR introduces Django 2.1 compatibility changes and black formatting. However, there are two critical bugs that will cause runtime failures: Python 3 incompatible base64 decoding and incorrect Django reverse() API usage.

2 issues detected:

🐞 Bug - Using string.decode("base64") which doesn't exist in Python 3 🛠️

Details: The base64 decoding method is incompatible with Python 3. In Python 3, strings don't have a decode() method. This will cause an AttributeError at runtime when HTTP Basic Authentication is used.
File: piston/authentication.py (60-60)
_{🛠️ A suggested code correction is included in the review comments.}

🐞 Bug - Passing fields as the second positional argument (urlconf) instead of as args or kwargs keyword argument 🛠️

Details: The reverse() function is being called with incorrect arguments. The second argument to reverse() is 'urlconf', not 'args'. The old code used permalink() which properly handled the args/kwargs distinction, but this direct call will fail or produce incorrect URLs.
File: piston/emitters.py (287-287)
_{🛠️ A suggested code correction is included in the review comments.}

Generated by LinearB AI and added by gitStream.
_{AI-generated content may contain inaccuracies. Please verify before using.
💡 Tip: You can customize your AI Review using Guidelines Learn how}

💡 Code Suggestions

piston/authentication.py:60 🐞 Bug - Python 3 Incompatible: Replace with: auth = base64.b64decode(auth.strip()).decode('utf-8') and add import base64 at the top of the file.

            auth = base64.b64decode(auth.strip()).decode('utf-8')

piston/emitters.py:287 🐞 Bug - Incorrect API Usage: Replace with: ret["resource_uri"] = reverse(url_id, args=fields) if fields is a tuple/list, or ret["resource_uri"] = reverse(url_id, kwargs=fields) if fields is a dict. Check what resource_uri() returns to determine the correct parameter name.

                        ret["resource_uri"] = reverse(url_id, args=fields)