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

bugfix: fill department description with jstl out tag (for v3.x branch) #1407

Merged
merged 1 commit into from
Oct 11, 2020

Conversation

honnel
Copy link
Contributor

@honnel honnel commented Oct 9, 2020

fixes #1399 in v3.x branch

@honnel honnel changed the title bugfix: fill department description with jstl out tag bugfix: fill department description with jstl out tag (for v3.x branch) Oct 9, 2020
@derTobsch derTobsch force-pushed the bugfix/department-description-output-1399-v3.x branch from dc8e8fc to 3329a8d Compare October 11, 2020 16:46
use `c:out` from jstl core for evaluation of department description
This couses excaping of xml tags instead of allow to manipulate html

* see: https://docs.oracle.com/javaee/5/jstl/1.1/docs/tlddocs/c/out.html
* closes: #1399
@derTobsch derTobsch force-pushed the bugfix/department-description-output-1399-v3.x branch from 3329a8d to b7a50cf Compare October 11, 2020 16:48
@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@derTobsch derTobsch merged commit e17afbc into v3.x Oct 11, 2020
@derTobsch derTobsch deleted the bugfix/department-description-output-1399-v3.x branch October 11, 2020 16:55
@derTobsch derTobsch added this to the 3.4.5 milestone Oct 30, 2020
@github-actions github-actions bot added the status: planned This issue is planned in a milestone and won't be touched by stale bot. label Oct 30, 2020
@honnel honnel added the type: security security issues label Nov 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
status: planned This issue is planned in a milestone and won't be touched by stale bot. type: security security issues
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants