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.

Sign up for GitHub

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

Jump to bottom

updated load_current_user in api to check user group in list #111

Merged
merged 1 commit into from
Feb 27, 2024
Merged

updated load_current_user in api to check user group in list #111

merged 1 commit into from
Feb 27, 2024

Commits on Feb 27, 2024

  1. updated load_current_user in api to check user group in list 

    Problem
- The API was not authenticating sanctuary users because
 load_current_user was only checking if the user group was medical

Solution
- Updated the load_current_user() function to check if the user group is in the list of allowed groups

Ticket
N/A

Tests
- docker compose up testapi
- manually logged in as a sanctuary user and checked frontend behaviour
    @critch646
    critch646 committed Feb 27, 2024
    Configuration menu
    Copy the full SHA
    a2dc3d8 View commit details
    Browse the repository at this point in the history