Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Gallery images on Facebook are tiny with PrivacyBadger enabled on both Chrome and Firefox #2214

Closed
Dyras opened this issue Nov 10, 2018 · 27 comments · Fixed by #2307 or #2501
Closed
Assignees
Labels
broken site bug first-party relating to first-party scripts

Comments

@Dyras
Copy link
Contributor

Dyras commented Nov 10, 2018

Greetings! For some reason, when you click a picture in the gallery under "Messenger" on Facebook, the pictures are always really small. This happens on both the latest Firefox and the latest Chrome, with the latest PrivacyBadger on both.

How to reproduce:

  1. Go on here https://www.facebook.com/messages/t
  2. Click a chat with an image you want to view
  3. Click the image in the bottom right corner, where all Facebook images in that chat are stored
  4. The image should be this "big":
    image
    Note how it's opened in a separate tab instead of inside the Messenger tab
  5. Deactivate PrivacyBadger on Facebook
  6. Click the same image again
  7. It's now this big:
    image
    Note how it's opened in the same tab as Messenger. Just on top of the chat.
@ghostwords
Copy link
Member

Hello and thank you for the bug report! I wonder if this is the same kind of issue as #2181 (comment), where we remove too much from the link element while removing tracking, although maybe not, need to take a look.

@ghostwords
Copy link
Member

@bcyphers Could you take a look please?

@ghostwords ghostwords added bug broken site first-party relating to first-party scripts labels Nov 12, 2018
@Dyras
Copy link
Contributor Author

Dyras commented Feb 14, 2019

@ghostwords @bcyphers Any news on this? It's been 3 months now. Facebook is a major website and this affects everyone using both PrivacyBadger and Facebook, and I'm getting pretty sick of it :(

@ghostwords
Copy link
Member

We should have a fix released next week: #2307.

@ghostwords
Copy link
Member

This should be fixed as of today's Privacy Badger update (version 2019.2.19).

@Dyras
Copy link
Contributor Author

Dyras commented Feb 20, 2019

@ghostwords Unless I'm missing something, this is still happening on Chrome.

Version info:
image

What the pictures look like when I click them:
image

What the pictures should look like when I click them:
image

@ghostwords
Copy link
Member

@bcyphers Could you take a look please?

@ghostwords ghostwords reopened this Feb 24, 2019
@ghostwords
Copy link
Member

Does this happen on Firefox for Android only, or is it also an issue on the desktop?

@Dyras
Copy link
Contributor Author

Dyras commented Feb 24, 2019

@ghostwords I'm using Chrome on Windows 10

@ghostwords
Copy link
Member

I see, I got confused because those screenshots are from Android.

@Dyras
Copy link
Contributor Author

Dyras commented Feb 24, 2019

@ghostwords Sorry, I should have been more clear :( I found the pictures on Reddit and sent them to someone

@bcyphers
Copy link
Contributor

Sorry for the delay on this, I'll take a look this week.

@Dyras
Copy link
Contributor Author

Dyras commented Mar 27, 2019

No problem! The fact that you guys are even responding to this speaks wonders for how great this project is.

@bcyphers
Copy link
Contributor

I'm having some trouble duplicating this. Can you share your chrome version number?

@Dyras
Copy link
Contributor Author

Dyras commented Apr 11, 2019

Version 73.0.3683.103

Do you want me to try it on a fresh PortableApp of Firefox or Chrome or something? I disabled all other Chrome extensions and it's still happening.

Edit: Here's how to do it
Go here: https://www.facebook.com/messages/t/
A chat should open, otherwise, click one.
Press a picture in the bottom right corner
The picture should be small

@bcyphers
Copy link
Contributor

Okay, I'm still having trouble replicating it on Ubuntu (work computer), but found something. If I left-click on a picture, the picture displays like normal (in the gallery view, full size). However, if I middle-click on the photo, or right-click -> "open in new tab," the photo opens in a new tab in tiny mode.

I can test this on Windows when I get home tonight.

Also, it seems to more consistently happen if you scroll up in a conversation, rather than sending new pictures and clicking them.

Is this a non-deterministic thing? If so, how often does it happen? are there circumstances where it never or always happens?

@Dyras
Copy link
Contributor Author

Dyras commented Apr 12, 2019

I deleted that comment because I couldn't reproduce that issue anymore.

However, the issue with the Shared Photos are still very reproducable on both MacOS and Windows 10.

Just click any sent message:
Skärmavbild 2019-04-12 kl  02 24 16

@bcyphers
Copy link
Contributor

bcyphers commented Apr 12, 2019

Okay, now I'm using Chrome 73.0.3683.103 on Windows 10. Still not seeing the issue when I left-click a picture in the bottom right corner, with Privacy Badger enabled. Also no issue when I click on an in-line image in the message history after scrolling up for a while. Everything works okay in Firefox as well.

Edit: Ah okay, just saw your comment above. I'll check on that too.

@bcyphers
Copy link
Contributor

Ugh, still not seeing anything. I've tried clicking on photos uploaded by both me and others in the chat, both recent and old, and everything opens up in gallery view properly. Would you mind trying in an instance of Chrome with a fresh user profile?

And just to check, you're using the latest version of PB (2019.2.19)?

@Dyras
Copy link
Contributor Author

Dyras commented Apr 12, 2019

I'm using 2019.2.19, yes.

I'm trying to reproduce it now. It was reproducable 100% of the time literally 20 minutes ago and now it's working as intended.

What the fuck :(

Will try to figure out what's causing it later. Maybe it's something that starts happening if you're on Facebook Messenger a really long time? I tend to have it open practically all day.

I'll try to figure it out tomorrow. I live in Europe so it's bedtime now :(

@bcyphers
Copy link
Contributor

okay! let me know here if you figure out how to repro. Definitely know the pain of the Heisenbug.

@Dyras
Copy link
Contributor Author

Dyras commented Apr 14, 2019

@bcyphers
I've managed to figure it out!

You have to set the Facebook language to Swedish (Svenska) and it will start happening immediately afterwards. I changed my language to English for the picture which is why I didn't realize this sooner.

Edit: German is also affected.

@Dyras
Copy link
Contributor Author

Dyras commented Jun 20, 2019

@bcyphers Any news on this? Just wondering, because I managed to figure out how to do it. Just switch to (seemingly) any non-English language on Facebook and it starts happening automagically.

@Dyras
Copy link
Contributor Author

Dyras commented Oct 24, 2019

@bcyphers @ghostwords this is still happening with your language set to Swedish :(

Any chance for some love?

@ghostwords
Copy link
Member

I'll take a look next week.

@ghostwords
Copy link
Member

OK, I see what's going on. @bcyphers We thought we fixed this issue with #2307, and we did, for Facebook locales where the ARIA label for photos is set to "photo". Apparently, this label is localized. What! Yes. For example, aria-label is set to "Foto" auf Deutsch.

@mgziminsky
Copy link

That makes sense, aria attributes are used for accessibility tools like screen readers, so they would need to be localized.

You may be able to whitelist a._3m31 instead, or a._3m31[aria-label] to be a little more explicit if you wanted. It's possible that class could change frequently though, but it's also possible it won't, and I don't see any other options that would be any more reliable.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
broken site bug first-party relating to first-party scripts
Projects
None yet
4 participants