Skip to content

(enhancement) - re-worked the Playground page to use CodeMirror#352

Open
catull wants to merge 1 commit intoschibsted:masterfrom
catull:feature/pimp-the-page
Open

(enhancement) - re-worked the Playground page to use CodeMirror#352
catull wants to merge 1 commit intoschibsted:masterfrom
catull:feature/pimp-the-page

Conversation

@catull
Copy link
Copy Markdown

@catull catull commented May 27, 2024

All former textareas and the result field are now CodeMirror editors with syntax highlighting.

Addresses #348.

All former textareas and the result field are now CodeMirror editors with syntax highlighting.
@catull catull changed the title (enhancement) - re-worked the HTML page to use CodeMirror (enhancement) - re-worked the Playground page to use CodeMirror May 27, 2024
@catull
Copy link
Copy Markdown
Author

catull commented May 27, 2024

I had to change the URL from jstl-demo to jstl.
You may have the two in use, locally it only works with one of them.

@catull
Copy link
Copy Markdown
Author

catull commented Sep 26, 2024

Hello @larsga. Can you please look into this PR? Thanks

response.getOutputStream().write(mapper.writerWithDefaultPrettyPrinter().writeValueAsBytes(output));
} catch (Exception e) {
try (PrintStream ps = new PrintStream(response.getOutputStream())) {
e.printStackTrace(ps);

Check warning

Code scanning / CodeQL

Information exposure through a stack trace Medium

Error information
can be exposed to an external user.
@catull catull force-pushed the feature/pimp-the-page branch from 989eb47 to fc32b0a Compare June 6, 2025 14:55
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.

2 participants