Opened 8 years ago

Closed 8 years ago

Last modified 8 years ago

#7503 closed defect (fixed)

the verge doesn't show comments

Reported by: cypherpunks Owned by: pde
Priority: Medium Milestone:
Component: HTTPS Everywhere/EFF-HTTPS Everywhere Version: HTTPS-E 4.0dev1
Severity: Keywords:
Cc: Actual Points:
Parent ID: #7851 Points:
Reviewer: Sponsor:

Description

  1. Go to http://www.theverge.com/2012/11/16/3655882/windows-8-pc-sales-reportedly-well-below-microsofts-internal-projections for example, with https everywhere enabled => Comments don't show up.
  1. Go to the button of https everywhere and disable the rules for this page - comments still disappeared!

Fix: Go to the button of https everywhere and click the option disable https everywhere (not need to be on the addons manager, on the extension itself) => Now the comments load.

Child Tickets

Change History (3)

comment:1 Changed 8 years ago by pde

This is a weird bug. The comments are hosted by SBNation, but it doesn't seem that disabling that ruleset (or even all of the rulesets active in the page) fixes the problem. That's true even after clearing cookies, to prevent securecookie confusion.

comment:2 Changed 8 years ago by pde

Resolution: fixed
Status: newclosed

Oh. There's actually a ruleset for The Verge itself, but it only affects XHR requests to the site, and those are not successfully associated with the window. From the logs:

{{{No window associated with request: http://www.theverge.com/posts/load_comments/3419923
Processing http://www.theverge.com/posts/load_comments/3419923
Notifying observers of rewrite from http://www.theverge.com/posts/load_comments/3419923 to https://www.theverge.com/posts/load_comments/3419923
Scheduling channel replacement for http://www.theverge.com/posts/load_comments/3419923
Ran channel replacement for http://www.theverge.com/posts/load_comments/3419923
No window associated with request: https://www.theverge.com/statsd_proxy/
Processing https://www.theverge.com/statsd_proxy/
No window associated with request: https://www.theverge.com/posts/load_comments/3419923
Processing https://www.theverge.com/posts/load_comments/3419923}}}

I am fixing this specific issue by disabling the ruleset, but this is a larger open bug.

comment:3 Changed 8 years ago by pde

Parent ID: #7851
Note: See TracTickets for help on using tickets.