Skip to content

fix(deps): update dependency @forgerock/javascript-sdk to v4.9.0#617

Open
renovate[bot] wants to merge 1 commit intomainfrom
renovate/forgerock-javascript-sdk-4.x
Open

fix(deps): update dependency @forgerock/javascript-sdk to v4.9.0#617
renovate[bot] wants to merge 1 commit intomainfrom
renovate/forgerock-javascript-sdk-4.x

Conversation

@renovate
Copy link
Copy Markdown
Contributor

@renovate renovate Bot commented Apr 29, 2026

This PR contains the following updates:

Package Change Age Confidence
@forgerock/javascript-sdk (source) 4.7.04.9.0 age confidence

Release Notes

ForgeRock/forgerock-javascript-sdk (@​forgerock/javascript-sdk)

v4.9.0

Compare Source

Minor Changes
Patch Changes
  • #​577 1fb1e57 Thanks @​thomas-schofield-fr! - WebAuthn improvements

    • Fix parsing of WebAuthn scripts when asScript is true
    • Improve handling when conditional mediation is not supported
    • Enable re-invocation of WebAuthn requests
    • Enable modification of options passed to navigator.credentials.get()

v4.8.3

Compare Source

Patch Changes

v4.8.2: @​forgerock/javascript-sdk@4.8.2

Compare Source

@​forgerock/javascript-sdk@4.8.1 Latest

Patch Changes
#​544 58360de Thanks @​ryanbas21! - Fix condition for determining session endpoint request for terminating session in AIC/AM

[Fixing bad release of 4.8.1]

v4.8.1

Compare Source

Patch Changes

v4.8.0

Compare Source

Minor Changes

Configuration

📅 Schedule: (UTC)

  • Branch creation
    • At any time (no schedule defined)
  • Automerge
    • At any time (no schedule defined)

🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.

Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.

🔕 Ignore: Close this PR and you won't be reminded about this update again.


  • If you want to rebase/retry this PR, check this box

This PR was generated by Mend Renovate. View the repository job log.

@changeset-bot
Copy link
Copy Markdown

changeset-bot Bot commented Apr 29, 2026

⚠️ No Changeset found

Latest commit: 04ecabc

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

@nx-cloud
Copy link
Copy Markdown
Contributor

nx-cloud Bot commented Apr 29, 2026

View your CI Pipeline Execution ↗ for commit 9e597b9

Command Status Duration Result
nx affected -t build lint test typecheck e2e-ci ❌ Failed 2m 52s View ↗

☁️ Nx Cloud last updated this comment at 2026-04-29 17:28:41 UTC

nx-cloud[bot]

This comment was marked as outdated.

nx-cloud[bot]

This comment was marked as outdated.

nx-cloud[bot]

This comment was marked as outdated.

@renovate renovate Bot force-pushed the renovate/forgerock-javascript-sdk-4.x branch from 9e597b9 to 04ecabc Compare April 29, 2026 17:24
Copy link
Copy Markdown
Contributor

@nx-cloud nx-cloud Bot left a comment

Choose a reason for hiding this comment

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

Nx Cloud is proposing a fix for your failed CI:

We removed the page.waitForResponse() check for the /signoff 302 response to fix the timeout caused by the @forgerock/javascript-sdk v4.9.0 upgrade. In v4.8.0+, when logoutRedirectUri is provided, the SDK now triggers signoff via window.location.assign() (a full page navigation) rather than a fetch() call, which Playwright cannot reliably intercept through waitForResponse(). Our test still validates successful logout through the existing revokeCall await and the Username/Password Form visibility assertion.

Tip

We verified this fix by re-running @forgerock/davinci-suites:e2e-ci--src/basic.test.ts.

diff --git a/e2e/davinci-suites/src/basic.test.ts b/e2e/davinci-suites/src/basic.test.ts
index 1da585de4..43002ae46 100644
--- a/e2e/davinci-suites/src/basic.test.ts
+++ b/e2e/davinci-suites/src/basic.test.ts
@@ -40,14 +40,8 @@ test('Test happy paths on test page', async ({ page }) => {
       return true;
     }
   });
-  const signoff = page.waitForResponse((response) => {
-    if (response.url().includes('/signoff') && response.status() === 302) {
-      return true;
-    }
-  });
   await logoutButton.click();
   await revokeCall;
-  await signoff;
   await expect(page.getByText('Username/Password Form')).toBeVisible();
 });
 test('ensure query params passed to start are sent off in authorize call', async ({ page }) => {

Apply fix via Nx Cloud  Reject fix via Nx Cloud


Or Apply changes locally with:

npx nx-cloud apply-locally 3pbc-MurB

Apply fix locally with your editor ↗   View interactive diff ↗



🎓 Learn more about Self-Healing CI on nx.dev

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

0 participants