Open Bug 1813066 Opened 2 years ago Updated 8 months ago

Remove ShouldResistFingerprinting call from nsIndexedToHTML.cpp

Categories

(Core :: Networking, task, P3)

task

Tracking

()

People

(Reporter: tschuster, Unassigned)

References

(Blocks 1 open bug)

Details

(Whiteboard: [necko-triaged])

This call was added in bug 1560574. I think this problem should now be resolved with the removal of FTP. I think nsIndexedToHTML is only used for privileged folders like file:// or resource:// now.

See Also: → 1560574
Flags: needinfo?(kershaw)

(In reply to Tom Schuster (MoCo) from comment #0)

This call was added in bug 1560574. I think this problem should now be resolved with the removal of FTP. I think nsIndexedToHTML is only used for privileged folders like file:// or resource:// now.

Yes, I think we can remove this call, since FTP is already removed.

Severity: -- → N/A
Flags: needinfo?(kershaw)
Priority: -- → P3
Whiteboard: [necko-triaged]
Assignee: nobody → tschuster
Depends on: 1824325

Yes, I think we can remove this call, since FTP is already removed.

Turns out that we still support application/http-index-format for regular web content, so this isn't true. I opened bug 1824325 to consider removing that support outside of file:// etc.

Assignee: tschuster → nobody

So now that Malte fixed bug 1824325 (thanks!) can anyone still think of a way of triggering this code for fingerprinting? I think this only works when embedding file:// URLs is allowed, which it definitely isn't for normal webpages.

You need to log in before you can comment on or make changes to this bug.