Skip to content

Conversation

@bodom0015
Copy link
Member

@bodom0015 bodom0015 commented May 29, 2025

Problem

CLEANDB-frontend prototype is currently publicly accessible

We would like this to be behind the same (temporary) auth wall that is currently used by OpenEnzymeDB

Approach

  • feat: adjust URL patterns to share cookies between CLEANDB and OED prototypes
  • feat: add private-auth/errors middleware to redirect without auth

TODOs:

  • add middleware to CLEANDB-api

How to Test

NOT CURRENTLY DEPLOYED

Staging:

  1. Close all incognito windows, and open a fresh incognito window
  2. Navigate to CLEANDB staging: https://cleandb.staging.openenzymedb.mmli1.ncsa.illinois.edu
    • You should be prompted to login in with Keycloak
  3. Log into Keycloak using CILogon or username/password
    • If your account is in the correct Keycloak group (openenzymedb), you should be routed to the CLEANDB frontend
    • If your account is not in the correct Keycloak group (openenzymedb), you should be shown a 403: Forbidden error page

Production:

  1. Close all incognito windows, and open a fresh incognito window
  2. Navigate to CLEANDB prod: https://cleandb.openenzymedb.platform.moleculemaker.org
    • You should be prompted to login in with Keycloak
  3. Log into Keycloak using CILogon or username/password
    • If your account is in the correct Keycloak group (openenzymedb), you should be routed to the CLEANDB frontend
    • If your account is not in the correct Keycloak group (openenzymedb), you should be shown a 403: Forbidden error page

@bodom0015
Copy link
Member Author

bodom0015 commented Jun 18, 2025

This PR should not change the Docker image build, so the following error does not affect this PR at all

However, I did notice a problem with the image build - filed this as a new issue:
#4

@bodom0015 bodom0015 marked this pull request as draft June 23, 2025 18:24
@bodom0015
Copy link
Member Author

bodom0015 commented Jun 23, 2025

Don't merge until next week 🙏 hopefully product is public by then

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.

2 participants