Skip to content

fix(@wdio/allure-reporter): Encode HTML entities#14723

Merged
christian-bromann merged 1 commit intowebdriverio:mainfrom
sventschui:allure-encode-html-entities
Sep 21, 2025
Merged

fix(@wdio/allure-reporter): Encode HTML entities#14723
christian-bromann merged 1 commit intowebdriverio:mainfrom
sventschui:allure-encode-html-entities

Conversation

@sventschui
Copy link
Contributor

Proposed changes

Encode HTML entities in the message / stack reported by the allure reporter as otherwise the allure report might completely break (i.e. when using testing-library that includes a stringified DOM tree in its error messages)

Types of changes

  • Polish (an improvement for an existing feature)
  • Bugfix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (improvements to the project's docs)
  • Specification changes (updates to WebDriver command specifications)
  • Internal updates (everything related to internal scripts, governance documentation and CI files)

Checklist

  • I have read the CONTRIBUTING doc
  • I have added tests that prove my fix is effective or that my feature works
  • I have added the necessary documentation (if appropriate)
  • I have added proper type definitions for new commands (if appropriate)

Backport Request

  • This change is solely for v9 and doesn't need to be back-ported
  • Back-ported PR at #XXXXX

Further comments

none

Reviewers: @webdriverio/project-committers

@linux-foundation-easycla
Copy link

linux-foundation-easycla bot commented Aug 25, 2025

CLA Signed

The committers listed above are authorized under a signed CLA.

  • ✅ login: sventschui / name: Sven (e7dec3c)

@sventschui sventschui force-pushed the allure-encode-html-entities branch from 31a303f to adf7bc0 Compare August 26, 2025 08:18
@sventschui sventschui force-pushed the allure-encode-html-entities branch from adf7bc0 to e7dec3c Compare August 26, 2025 08:21
Copy link
Member

@christian-bromann christian-bromann left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@christian-bromann christian-bromann added the PR: Bug Fix 🐛 PRs that contain bug fixes label Sep 9, 2025
@christian-bromann christian-bromann merged commit 1e2ecd2 into webdriverio:main Sep 21, 2025
90 of 92 checks passed
@wdio-bot
Copy link
Contributor

Hey sventschui 👋

Thank you for your contribution to WebdriverIO! Your pull request has been marked as an "Expensable" contribution.

We've sent you an email with further instructions on how to claim your expenses from our development fund.
Please make sure to check your spam folder as well. If you have any questions, feel free to reach out to us at expense@webdriver.io or in the contributing channel on Discord.

We are looking forward to more contributions from you in the future 🙌

Have a nice day,
The WebdriverIO Team 🤖

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

Labels

Expensable $50 💸 PR: Bug Fix 🐛 PRs that contain bug fixes

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants