Skip to content

feat: update anvil cmg auth config to use ecm endpoint (#4694)#4696

Merged
NoopDog merged 2 commits intomainfrom
cc/4693-ecm-endpoint
Mar 12, 2026
Merged

feat: update anvil cmg auth config to use ecm endpoint (#4694)#4696
NoopDog merged 2 commits intomainfrom
cc/4693-ecm-endpoint

Conversation

@frano-m
Copy link
Copy Markdown
Contributor

@frano-m frano-m commented Mar 12, 2026

Closes #4694.

This pull request updates dependencies and modifies service endpoint URLs for authentication in both development and production configurations. The most important changes are grouped below:

Dependency update:

  • Upgraded the @databiosphere/findable-ui dependency version in package.json from ^49 to ^49.4.1, which may include bug fixes or new features.

Service endpoint changes:

  • Changed the nihStatus endpoint in site-config/anvil-cmg/dev/authentication/constants.ts to use the new RAS OAuth endpoint (https://externalcreds.dsde-dev.broadinstitute.org/api/oauth/v1/ras) instead of the previous Firecloud Orchestration endpoint.
  • Changed the nihStatus endpoint in site-config/anvil-cmg/prod/authentication/constants.ts to use the new RAS OAuth endpoint (https://externalcreds.dsde-prod.broadinstitute.org/api/oauth/v1/ras) instead of the previous Firecloud Orchestration endpoint.

Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates Anvil CMG authentication configuration to use the External Credentials Manager (ECM) / RAS OAuth endpoint and bumps the @databiosphere/findable-ui dependency to a newer v49 patch release.

Changes:

  • Upgrade @databiosphere/findable-ui from ^49 to ^49.4.1.
  • Switch Anvil CMG dev nihStatus endpoint from Firecloud Orchestration to the ECM/RAS OAuth endpoint.
  • Switch Anvil CMG prod nihStatus endpoint from Firecloud Orchestration to the ECM/RAS OAuth endpoint.

Reviewed changes

Copilot reviewed 3 out of 4 changed files in this pull request and generated 1 comment.

File Description
site-config/anvil-cmg/prod/authentication/constants.ts Updates prod nihStatus service endpoint to ECM/RAS.
site-config/anvil-cmg/dev/authentication/constants.ts Updates dev nihStatus service endpoint to ECM/RAS.
package.json Bumps @databiosphere/findable-ui version range to ^49.4.1.
package-lock.json Aligns lockfile to findable-ui@49.4.1 resolution.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment on lines 18 to 20
nihStatus:
"https://firecloud-orchestration.dsde-prod.broadinstitute.org/api/nih/status",
"https://externalcreds.dsde-prod.broadinstitute.org/api/oauth/v1/ras",
profile: "https://sam.dsde-prod.broadinstitute.org/register/user/v1",
Copy link

Copilot AI Mar 12, 2026

Choose a reason for hiding this comment

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

This PR updates anvil-cmg dev/prod nihStatus to the externalcreds RAS endpoint, but site-config/anvil-cmg/tempdev/authentication/constants.ts still hardcodes the old Firecloud Orchestration /api/nih/status URL. If tempdev is still a supported deployment target, consider updating/removing that stale endpoint to avoid confusion or accidental regressions later.

Copilot uses AI. Check for mistakes.
@NoopDog NoopDog merged commit ed2262a into main Mar 12, 2026
7 checks passed
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.

Update AnVIL CMG auth config to use External Credentials Manager endpoint

3 participants