Skip to content

feat: switch to ibiofoundry hostname#155

Open
bodom0015 wants to merge 3 commits intomainfrom
lambert8/feature/update-to-ibiofoundry-hostname
Open

feat: switch to ibiofoundry hostname#155
bodom0015 wants to merge 3 commits intomainfrom
lambert8/feature/update-to-ibiofoundry-hostname

Conversation

@bodom0015
Copy link
Member

@bodom0015 bodom0015 commented Jul 2, 2025

Problem

We would like to serve a copy of this page at clean.platform.ibiofoundry.illinois.edu

Approach

  • wip: serve at ibiofoundry host, redirect others to ibiofoundry (see comments for next steps)

DO NOT MERGE until the DNS rule has been set up for *.platform.ibiofoundry.illinois.edu DNS rule has been created 👍

TODOs

  • New OAuth2 proxy instance needed to manage the new domain *.platform.ibiofoundry.illinois.edu
  • Add new domain to existing CORS rules throughout (maybe utilize wildcards more frequently in CORS rules)
  • need to revisit CLEAN job success/failed emails, which may no longer contain the correct hostname
  • New Keycloak realm?

Considerations

With this approach, the URL would always say clean.platform.ibiofoundry.illinois.edu

The following hosts will redirect to this new hostname:

  • clean.platform.moleculemaker.org -> clean.platform.ibiofoundry.illinois.edu
  • clean.frontend.mmli1.ncsa.illinois.edu -> clean.platform.ibiofoundry.illinois.edu

How to Test

  1. Navigate to https://clean.platform.ibiofoundry.illinois.edu/configuration
    • You should see the familiar CLEAN app
  2. Navigate to https://clean.platform.moleculemaker.org/configuration
    • You should see the same familiar CLEAN app
    • NOTE: this is a single container of clean-frontend running with 2 separate hostnames, and currently both hostnames should be allowed to be used
    • Also noting that this case should actually FAIL, but for some reason right now it is succeeding.. which leaves me with other questions 😅

@bodom0015 bodom0015 requested a review from matthewberry July 2, 2025 20:09
@bodom0015 bodom0015 marked this pull request as draft July 2, 2025 20:10
@bodom0015 bodom0015 marked this pull request as ready for review September 22, 2025 17:14
@bodom0015 bodom0015 changed the title [WIP] feat: switch to ibiofoundry hostname feat: switch to ibiofoundry hostname Sep 22, 2025
@bodom0015
Copy link
Member Author

Reminder to update these rules to work the same way for mmli2 🙏

@bodom0015
Copy link
Member Author

bodom0015 commented Dec 19, 2025

Double-checked and this has been deployed to production for awhile 👍 will deploy this to mmli2 when we're ready

Also confirmed that the changes currently running on production are reflected in this PR

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.

1 participant