Sorry @rafgia I should have picked this up before, but you'll see that the tests are still failing, now with the following error:

...
  File "/env3/lib/python3.9/site-packages/django/template/base.py", line 600, in compile_filter
    return FilterExpression(token, self)
  File "/env3/lib/python3.9/site-packages/django/template/base.py", line 703, in __init__
    raise TemplateSyntaxError(
django.template.exceptions.TemplateSyntaxError: Could not parse the remainder: '()' from 'request.build_absolute_uri()'

The problem is that function calls inside a template do not require the parentheses (). I can't find where this is explained in the Django documentation, but you'll see the function that you are replacing doesn't include parentheses.

(Side note, you can run the tests locally with python manage.py test, helping to pick up problems before you open the pull request). Before running tests, you generally want to make sure that you have a fresh database by running: python manage.py resetdb followed by python manage.py loaddemo.

Thanks Raffaele, looks good and tests are now passing. Before we merge, please could you squash your commits? You could do this with:

# reset to the top of the dev branch
git reset bea5f64838c6d7a71d5610302c1c74e9921fb0e4

# add and commit the file again
git add physionet-django/templates/404.html
git commit -m "Fix deprecated call to HttpRequest.get_raw_uri. Closes #2296"

# Force push (forcing because you need to overwrite the commit history of your branch)
git push --force

Looks great, thanks Raffaele!