Skip to content

[1.0.x] Handle malformed Unicode in generated avatars - #137

Merged
ThaminduDilshan merged 3 commits into
thunder-id:1.0.xfrom
NipuniBhagya:1.0.x-fixes
Oct 26, 2026
Merged

ThaminduDilshan merged 3 commits into
thunder-id:1.0.xfrom
NipuniBhagya:1.0.x-fixes

Conversation

@NipuniBhagya

Copy link
Copy Markdown
Contributor

Purpose

Backports the fix for malformed Unicode in generated avatars (#113) to the 1.0.x maintenance branch.

Approach

Cherry-picks 974c486 from #113 unchanged with git cherry-pick -x. The original author is kept. The change touches only @thunderid/browser: generateAvatarDataUri.ts and its test.

When releasing, publish @thunderid/browser together with the SDKs that depend on it (react, vue and the framework SDKs built on them), so their users get the fixed browser version.

Related Issues

  • N/A

Related PRs

Checklist

  • Followed the contribution guidelines.
  • Manual test round performed and verified. @thunderid/browser builds on this branch and generateAvatarDataUri.test.ts passes (27 tests).
  • Documentation provided. (Add links if there are any)
  • Tests provided. (Add links if there are any)
    • Unit Tests
    • Integration Tests
  • Breaking changes. (Fill if applicable)
    • Breaking changes section filled.
    • breaking change label added.
  • Cross-SDK parity. Exactly one of parity/prs-raised or parity/prs-not-needed added.
    • If parity/prs-raised, the port links are posted as a reply on the parity check's comment.

Security checks

  • Followed secure coding standards.
  • Confirmed that this PR doesn't commit any keys, passwords, tokens, usernames, or other secrets.

🤖 Generated with Claude Code

(cherry picked from commit 974c486)
Signed-off-by: NipuniBhagya <nipunib@wso2.com>
@coderabbitai

coderabbitai Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review: 1
  • main

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

Run configuration

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@github-actions

github-actions Bot commented Oct 26, 2026 •

Copy link
Copy Markdown

🔀 Cross-SDK feature parity, no ports needed

This pull request is labelled parity/prs-not-needed: nothing here has to ship in the other ThunderID SDKs. If that changes, swap the label for parity/prs-raised.

@NipuniBhagya NipuniBhagya added the parity/prs-not-needed Nothing here reaches another SDK label Oct 26, 2026
Fixes GHSA-cg2g-xh64-v4q7 (critical), GHSA-mrv2-3pqr-j5jr (high) and lower
severity DOMPurify advisories. @thunderid/react and @thunderid/vue pin
dompurify exactly via the catalog, so 1.0.x releases would otherwise ship a
vulnerable version. Matches the version already on main.

Signed-off-by: NipuniBhagya <nipunib@wso2.com>
Makes `pnpm audit --audit-level=high` pass on the 1.0.x maintenance branch.

- Raise the existing body-parser, brace-expansion, sharp and shell-quote pins
  to main's versions, add main's devalue and source-map-js overrides, and port
  its justified audit ignores (node-forge, simple-git).
- Pin launch-editor, vite 6/7/8, proxy-addr, seroval, undici 7/8, vue and zod
  to their patched releases. Main picked most of these up through lockfile
  refreshes; here they are pinned so the lockfile doesn't drift.
- Bump packages/nextjs's `next` devDependency to 15.5.28 (as on main) and the
  Next.js samples' floor to ^15.5.28.

All of these reach the workspace through devDependencies, the samples or the
E2E suite, so they don't change what the published SDKs install.

Signed-off-by: NipuniBhagya <nipunib@wso2.com>
@ThaminduDilshan
ThaminduDilshan merged commit 8d2f33d into thunder-id:1.0.x Oct 26, 2026
7 checks passed
senthalan pushed a commit to thunder-id/thunderid that referenced this pull request Oct 27, 2026
Bump @thunderid/react and @thunderid/react-router to 1.0.9 to pick up the
SDK fix from thunder-id/javascript-sdks#137.
darshanasbg pushed a commit to thunder-id/thunderid that referenced this pull request Oct 28, 2026
Bump @thunderid/react and @thunderid/react-router to 1.0.9 to pick up the
SDK fix from thunder-id/javascript-sdks#137.

Signed-off-by: NipuniBhagya <nipunib@wso2.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

parity/prs-not-needed Nothing here reaches another SDK

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants