Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

FISH-10521 Escape HTTP Characters in REST Interface #7216

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

Pandrex247
Copy link
Member

@Pandrex247 Pandrex247 commented Feb 25, 2025

Description

Escapes HTTP characters in the REST interface, to help lock down any XSS attempts someone may try if they have access to the application server's filesystem.

Most config which would be displayed here is protected by validation of the domain.xml, as that prevents you from having the "<" or ">" characters as values. The key files however don't have this validation, and so we must escape them.

Important Info

Blockers

None

Testing

New tests

None

Testing Performed

On a clean domain:

Testing Environment

Windows 11, Maven 3.9.9, Zulu JDK 11.0.26

Documentation

payara/Payara-Documentation#552

Notes for Reviewers

None

Fix for CVE-2025-1534.

Uses the existing method to hopefully escape only the config values being displayed,
and not any of the valid HTTP tags.

Signed-off-by: Andrew Pielage <pandrex247@hotmail.com>
@Pandrex247 Pandrex247 force-pushed the FISH-10521-Escape-http-characters branch from 3818bb6 to 6e752a9 Compare February 25, 2025 14:34
@Pandrex247 Pandrex247 marked this pull request as ready for review February 25, 2025 15:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant