mozilla/browserid — comment on commit 591c3404789394bf68dccce2eb36c3cf819f355c
File: lib/browserid/views.js line 94
By ozten:

I know it is outside of the scope of this pull request, but...

Why do we have a useJavascript to control outputing script tags? I think progressive enhancement just takes care of this. Many browsers with JS disabled will not fetch those scripts (if bandwidth is the concern).

Another potential problem, is we don't Vary on JS support. So if the first request for a page was from a user with JS disabled, zues would cache the HTML without JS and serve that up to users who do have JS enabled. (Un-tested hypothesis).

Not a big deal, I am just curious. If we want to change this, I can file a bug.