Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Xmerl: Changed default values to disable XXE vulnerabilities #8438

Merged

Conversation

lthor
Copy link
Contributor

@lthor lthor commented Apr 30, 2024

The default values in xmerl permitted XXE vulnerabilities and are therefore changed to more restrictive ones.

  • xmerl_scan: default value for allow_entities is now 'false'
  • xmerl_sax_parser: default value for external_entities is now 'none'

xmerl_scan: default value for allow_entities is
      now false
xmerl_sax_parser: default value for external_entities
      is now none
@lthor lthor self-assigned this Apr 30, 2024
@lthor lthor added team:PS Assigned to OTP team PS testing currently being tested, tag is used by OTP internal CI bug Issue is reported as a bug labels Apr 30, 2024
Copy link
Contributor

github-actions bot commented Apr 30, 2024

CT Test Results

    2 files     24 suites   7m 42s ⏱️
2 162 tests 2 154 ✅  8 💤 0 ❌
3 980 runs  3 946 ✅ 34 💤 0 ❌

Results for commit 2ea54df.

♻️ This comment has been updated with latest results.

To speed up review, make sure that you have read Contributing to Erlang/OTP and that all checks pass.

See the TESTING and DEVELOPMENT HowTo guides for details about how to run test locally.

Artifacts

// Erlang/OTP Github Action Bot

@lthor lthor merged commit b5ad065 into erlang:master May 3, 2024
17 checks passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
bug Issue is reported as a bug team:PS Assigned to OTP team PS testing currently being tested, tag is used by OTP internal CI
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant