8000 Clarify that html in translations is not considered a security issue by carakas · Pull Request #3270 · forkcms/forkcms · GitHub
[go: up one dir, main page]

Skip to content

Clarify that html in translations is not considered a security issue#3270

Merged
carakas merged 2 commits intomasterfrom
carakas-patch-1
Feb 9, 2021
Merged

Clarify that html in translations is not considered a security issue#3270
carakas merged 2 commits intomasterfrom
carakas-patch-1

Conversation

@carakas
Copy link
Member
@carakas carakas commented Jan 8, 2021

Type

  • Enhancement

Pull request description

We sometimes get reports of people reporting that they found a security issue with our translation system. By adding this to the readme we are clarifying what we consider a feature instead of a security issue

@carakas carakas added this to the 5.9.3 milestone Jan 8, 2021
@carakas carakas requested a review from a team January 8, 2021 10:42
@codecov
Copy link
codecov bot commented Jan 8, 2021

Codecov Report

Merging #3270 (0a54d1c) into master (2f012b1) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master    #3270   +/-   ##
=========================================
  Coverage     27.79%   27.79%           
  Complexity     8010     8010           
=========================================
  Files           568      568           
  Lines         30564    30564           
=========================================
  Hits           8496     8496           
  Misses        22068    22068           
Flag Coverage Δ Complexity Δ
functional 23.73% <ø> (ø) 0.00 <ø> (ø)
installer 3.86% <ø> (ø) 0.00 <ø> (ø)
unit 7.63% <ø> (ø) 0.00 <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2f012b1...0a54d1c. Read the comment docs.

@carakas carakas merged commit 38f5836 into master Feb 9, 2021
@carakas carakas deleted the carakas-patch-1 branch February 9, 2021 10:07
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

0