Lighthouse has a new layout. Prefer the old one? Return to the old layout, and switch back any time from the link at the top of each page.

There was a problem

You must be a member of this account.

rhino xhr does not use contentType;charset or other content sniffing

#104

In env-js/src/platform/rhino @ line 251

    xhr.responseText = java.nio.charset.Charset.forName("UTF-8").
        decode(java.nio.ByteBuffer.wrap(baos.toByteArray())).toString()+"";

will either explode with an exception or produce junk if the doco is in latin-1 (or equiv)

First step is to write a test to prove this. then ideally follow a basic charset sniffer algorithm:

  • use content-type;charset=
  • if html, look for meta tag
  • look at byte-order marks
  • default to utf8

there are more complicated versions to do this, but this should cover the basics

In a different note, a bit above (line 219)

   var contentEncoding = connection.getContentEncoding() || "utf-8",

should just be || '', the content-encoding isn't used for charset encoding.

Reported by nickg · March 15th, 2010 @ 02:33 AM

State: new
Milestone: none
Assigned to: nobody

Activity

No comments yet.

Please Sign in or create a free account to add a new ticket.

With your very own profile, you can contribute to projects, track your activity, watch tickets, receive and update tickets through your email and much more.