Closed jonathanKingston closed 7 years ago
I believe this is a combination of first party as third party #110 and also our recent change from using response.originUrl
instead of response.documentUrl
in PR #155 . The latter can give us a third party (let's call it Site A) as the origin URL linking to another third party (Site B). If Site A is on the allowlist for its original first party site (and is therefore hidden), we can see these detached nodes.
My initial hunch as I mentioned on slack is that the code there didn't make anything worse due to documentUrl actually being wrong for certain cases but it perhaps just appeared less wrong.
Perhaps the assumption of non visible tabs are a third party should be questioned more.
I agree it was the right decision to change to originUrl
, we now just need to make sure we handle third parties making third party requests in the setThirdParty
logic.
@biancadanforth I'm still testing my patch however I think it solves a lot of the issues we have been having here.
Should be fixed with latest patch.
I'm seeing when loading:
This gets the following:![image](https://user-images.githubusercontent.com/338988/29796638-25b9892a-8c07-11e7-94ed-1dbf52d57cc4.png)
So for some reason we are seeing third parties having third parties which isn't expected. We then are not rendering third->third parties which maybe we should do anyway (not that the data should ever do this - this could be part of the ft+tp fix).