Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Quick: fix impersonation url #902

Merged
merged 4 commits into from
Nov 21, 2023
Merged

Quick: fix impersonation url #902

merged 4 commits into from
Nov 21, 2023

Conversation

rstijerina
Copy link
Member

@rstijerina rstijerina commented Nov 15, 2023

Overview

The django-impersonate middleware was not functioning properly with the url structure we had in place for some reason.

Testing

  1. Login in one tab with your user, and grant yourself superuser status
  2. Login in an incognito tab with another test user
  3. Go to https://cep.test/core/admin/impersonate/2 and confirm impersonation works

Copy link

codecov bot commented Nov 15, 2023

Codecov Report

Merging #902 (95c0767) into main (a181f6a) will increase coverage by 0.00%.
The diff coverage is 100.00%.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #902   +/-   ##
=======================================
  Coverage   63.40%   63.40%           
=======================================
  Files         432      432           
  Lines       12403    12404    +1     
  Branches     2585     2585           
=======================================
+ Hits         7864     7865    +1     
  Misses       4327     4327           
  Partials      212      212           
Flag Coverage Δ
javascript 69.75% <ø> (ø)
unittests 56.94% <100.00%> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files Coverage Δ
server/portal/urls.py 100.00% <100.00%> (ø)

Copy link
Collaborator

@chandra-tacc chandra-tacc left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Testing now with local user

Copy link
Collaborator

@jarosenb jarosenb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, same principle as what's been working in Designsafe.

@chandra-tacc chandra-tacc merged commit c97ce0c into main Nov 21, 2023
6 checks passed
@chandra-tacc chandra-tacc deleted the fix/impersonate-url branch November 21, 2023 16:44
chandra-tacc pushed a commit that referenced this pull request Dec 8, 2023
* fix impersonation url

* explicit impersonate urls; preserve previous impersonate path
chandra-tacc pushed a commit that referenced this pull request Dec 8, 2023
* fix impersonation url

* explicit impersonate urls; preserve previous impersonate path
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants