Opened 12 years ago
Closed 12 years ago
#10089 closed Bug (fixed)
<script> tag is not filtered out
Reported by: | Wiktor Walc | Owned by: | Piotrek Koszuliński |
---|---|---|---|
Priority: | Normal | Milestone: | CKEditor 4.1 RC |
Component: | General | Version: | |
Keywords: | Cc: |
Description
Thanks to #9829 CKEditor now creates HTML content which contains only allowed HTML tags.
The problem is that the <script>
tag can be still inserted in source mode and is not properly removed, even though there isn't any rule that would allow it (in editor.filter.allowedContent
). So it looks like this tag has been somehow forgotten.
I believe we should:
a) remove the
<script>
tag completely by default, leaving it to the developer to specify it inconfig.extraAllowedContent
OR
b) eventually specify it in
config.extraAllowedContent
by default. However ifconfig.extraAllowedContent
is set to an empty string, the tag should be still removed.
Also in case of <script>
tag we should simply remove the tag if its forbidden, without leaving the inner content in the editor.
Change History (6)
comment:1 Changed 12 years ago by
Milestone: | → CKEditor 4.1 |
---|
comment:2 Changed 12 years ago by
Status: | new → confirmed |
---|
comment:3 Changed 12 years ago by
Owner: | set to Piotrek Koszuliński |
---|---|
Status: | confirmed → assigned |
comment:4 Changed 12 years ago by
Status: | assigned → review |
---|
comment:5 Changed 12 years ago by
Status: | review → review_passed |
---|
comment:6 Changed 12 years ago by
Resolution: | → fixed |
---|---|
Status: | review_passed → closed |
Merged to major with git:f35bd1f on dev and b9e9dd8 on tests.
Pushed t/10089 on dev and tests.
<(no)script>
tags are by default removed, but may be passed if added to(extra)AllowedContent
.