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

Disable url previews in E2E rooms by default #2084

Closed
ara4n opened this issue Sep 4, 2016 · 11 comments
Closed

Disable url previews in E2E rooms by default #2084

ara4n opened this issue Sep 4, 2016 · 11 comments

Comments

@ara4n
Copy link
Member

ara4n commented Sep 4, 2016

No description provided.

@uhoreg
Copy link
Member

uhoreg commented Jan 23, 2017

In addition to disabling URL previews by default, the room setting to enable/disable URL previews should come with a warning in e2e rooms that enabling them may leak information.

@PureTryOut
Copy link

This happens in WhatsApp as well and is gaining a fair bit of attention on /r/Android. Please disable it by default, it shouldn't be that hard to do.

@ara4n ara4n added P1 and removed P2 labels Jun 18, 2018
@t3chguy
Copy link
Member

t3chguy commented Jun 19, 2018

@ara4n given the prio bump could you clarify the desired behaviour

  • Do we split the UserSettings toggle into non-e2ee and e2ee
  • Do we ignore account&default values for e2ee rooms and require it to be explicitly set per-room
  • something else

@turt2live
Copy link
Member

It makes sense to me to say that the room setting means nothing in e2e rooms, likewise for account. room-account or room-device (technically not settable) would have to be explicitly required.

@t3chguy
Copy link
Member

t3chguy commented Jun 19, 2018

so the latter option but also ignoring room

@Half-Shot
Copy link
Member

Explicitly set per room seems obvious to me PLUS making sure the user knows the implications (I realise they should know the implications as it's E2E, but I'd spell out the consequences anyway)

@t3chguy
Copy link
Member

t3chguy commented Jun 19, 2018

(I realise they should know the implications as it's E2E, but I'd spell out the consequences anyway)

This either way :)

@t3chguy
Copy link
Member

t3chguy commented Jun 21, 2018

image

opinion on wording? (ignore i18n warning)

@turt2live
Copy link
Member

I'd recommend starting with explaining what they are (which should really be included in every room to be honest). How does something like this sound?

When someone puts a URL in their message, a URL preview can be shown to give more information about that link such as the title, description, and an image from the website. In encrypted rooms, like this one, URL previews are disabled by default to ensure that your homeserver (where the previews are generated) cannot gather information about links you see in this room.

@t3chguy
Copy link
Member

t3chguy commented Jun 21, 2018

i like it

many thanks @turt2live

@t3chguy
Copy link
Member

t3chguy commented Jun 25, 2018

🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

7 participants