You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@manifoldcf.apache.org by "Karl Wright (JIRA)" <ji...@apache.org> on 2015/06/24 16:34:09 UTC

[jira] [Commented] (CONNECTORS-1215) Tag parsing in script tags shouldn't recognize "<" but does

    [ https://issues.apache.org/jira/browse/CONNECTORS-1215?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14599497#comment-14599497 ] 

Karl Wright commented on CONNECTORS-1215:
-----------------------------------------

The issue is complex because according to spec the code is doing the right thing.  Typically, <script> blocks look something like this:

{code}
<script ...>
<!--

...

//-->
</script>
{code}

The reason for the comment area is because without it, tags within the script block are supposed to be recognized as such, even if they are ignored.  Within comments, this does not happen, of course, which is why comments are used.

I don't believe it is a real standard, but some browsers try to interpret script blocks differently even when no comment is given.  We can try to emulate that behavior but it is likely that our emulation will not work for all web pages, since it's not a standard.  Exploring how this works on various browsers would be the first step.  Specifically, if you do something like this:

{code}
<script ...>

foo = "<script></script>";
bar = "hello";

</script>
{code}


... what happens?  Does the script end at the first </script>, or the second?  And, in what browsers?

Until we get more clarity it's going to be hard to do a feature that actually helps rather than hurts...

> Tag parsing in script tags shouldn't recognize "<" but does
> -----------------------------------------------------------
>
>                 Key: CONNECTORS-1215
>                 URL: https://issues.apache.org/jira/browse/CONNECTORS-1215
>             Project: ManifoldCF
>          Issue Type: Bug
>          Components: Web connector
>    Affects Versions: ManifoldCF 1.9, ManifoldCF 2.1
>            Reporter: Karl Wright
>            Assignee: Karl Wright
>             Fix For: ManifoldCF 1.10, ManifoldCF 2.2
>
>
> User says:
> "There appears to be a bug in the TagParseState when the comparison operator '<'  is encountered in the body of  a script tag.  It appears to get flagged as an open tag and then the next '</' closes it.  In my case, the next '</' is the script tag.  The ScriptParseState chomps everything until it encounters a second </script> tag.
> A live link that demonstrates this bug is here:
> http://www.prnewswire.com/search-results/news/Google%252C%2520Inc.-30-days-page-1-pagesize-20
> The '<' near line 2826 in the script body that begins near   line 2759 begins a new tag 'arraykeywords.length' which gets closed by the '</' in the closing script tag.  The ScriptParseState chomps all the html until it sees the end script tag near line 3385.
> At the moment, I'm not sure of a solution other than pushing the script tag handling up to the TagParseState and treating it like CDATA is."



--
This message was sent by Atlassian JIRA
(v6.3.4#6332)