Skip to content

Conversation

@faisalsiddique4400
Copy link
Contributor

fix(admin-ui): lock cedarling_wasm to v1.7.0 and improve Cedar init fallback handling (#2204)

🐞 Summary

This PR resolves a critical issue where the Admin UI fails to initialize the Cedarling engine due to incompatible changes in version 1.9.0 of the @janssenproject/cedarling_wasm package.


✅ Changes

  • Locked @janssenproject/cedarling_wasm to version 1.7.0 to avoid breaking behavior introduced in 1.9.0.
  • Added retry mechanism that attempts to initialize Cedarling up to 10 times.
  • Gracefully handles failure with a safe fallback if initialization continues to fail.

🧱 Notes

  • Prevents hard crashes in restricted or unstable environments (e.g., Solo).
  • Ensures smoother startup experience and consistent UI behavior regardless of Cedarling state.
  • Long-term solution will require coordination with upstream package maintainers.

🔗 Ticket

Closes: #2204
Title: Lock cedarling_wasm to v1.7.0 and improve Cedar init fallback handling

…allback handling (#2204)

Pinned cedarling_wasm to version 1.7.0 due to breaking changes in 1.9.0 that disrupt engine initialization.
Also added retry logic with a fallback mechanism after 10 unsuccessful attempts.
@faisalsiddique4400 faisalsiddique4400 removed the request for review from syntrydy July 25, 2025 12:23
@mo-auto
Copy link
Member

mo-auto commented Jul 25, 2025

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

@mo-auto mo-auto added comp-admin-ui Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality labels Jul 25, 2025
@sonarqubecloud
Copy link

@moabu moabu merged commit 787d0d6 into main Jul 25, 2025
8 checks passed
@moabu moabu deleted the admin-ui-issue-2204 branch July 25, 2025 13:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

comp-admin-ui Component affected by issue or PR kind-bug Issue or PR is a bug in existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants