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

Hyperlink previews #5724

Closed
2 tasks done
rotojp opened this issue Sep 27, 2016 · 16 comments
Closed
2 tasks done

Hyperlink previews #5724

rotojp opened this issue Sep 27, 2016 · 16 comments
Labels

Comments

@rotojp
Copy link

rotojp commented Sep 27, 2016

I have:

Bug description

When sending hyperlinks, Signal currently displays no previews.
This may be a feature lowering adaptation of Signal, as with #5445.

Screenshots

Google Allo

google-allo

Telegram

download

@2-4601 2-4601 changed the title [Feature Request] Hyperlink previews Hyperlink previews Sep 29, 2016
@2-4601 2-4601 added the feature label Sep 29, 2016
@TimesEnemy
Copy link

Greets.

I would rather a feature like this be optional and off by default. It will just generate more traffic and create unnecessary potential security holes. Perhaps when a link is created or detected by Signal, Signal could offer a checkbox or option to include a hyperlink preview into the mix.

@kmindi
Copy link

kmindi commented Sep 29, 2016

I agree with @TimesEnemy that this is at least a privacy issue and can also be a security issue concerning automatic loading of content from literally any source.

@kmindi kmindi mentioned this issue Sep 29, 2016
4 tasks
@2-4601
Copy link
Contributor

2-4601 commented Sep 29, 2016

WhatsApp does this on the client-side and only for the sender. When the sender types a link in the compose box WA detects it and fetches a preview.

@altring
Copy link

altring commented Sep 30, 2016

Desktop version also needs this :)

@Trolldemorted
Copy link
Contributor

Having the sender generate the preview is reasonable, i guess. A checkbox (that remembers the previous choice) would be great, as i'd propably never want to use this feature.

@E3V3A
Copy link

E3V3A commented Oct 5, 2016

Preview is a horrible privacy killer. Why even bother to use Signal if you want this? Just use Fb messenger or WhatsApp. Please close issue as N/A.

@rmueller83
Copy link

At least in iMessage this feature sucks ;-) http://rsmck.co.uk/blog/imessage-preview/

@oittaa
Copy link

oittaa commented Oct 11, 2016

@E3V3A what's wrong with the approach proposed in this thread?

  1. Make the feature optional.
  2. Sender device (which obviously trusts the the link more or less) fetches a preview and packs it in the message.

To me that looks like a zero privacy issue for the recipient. The current model sucks from usability perspective.

  1. Take a screenshot from a webpage.
  2. Attach the screenshot to a message.
  3. Add the correct URL to the message.

Only the last step should be needed in a modern messenger.

@l0rn
Copy link

l0rn commented Oct 28, 2016

i like the proposal of @oittaa . Eleminate the privacy issue by only letting the sender device download stuff and put the result in a message. I'd like to see that feature for random cat content gif urls I paste in the text field. Like Telegram does, but with only let my device download it.

@mderazon
Copy link

This is one of my favorite features in other IMs. I see some comments about security / privacy issues but they are not giving any examples. I can't think of any issue with that and i'm interested to know.

Actually, I find myself often not needing to open the link because I already know what the link is about so if anything, it's a 👍 for privacy / security :-)

Anyway, it's a huge plus for usability IMO.

@snaggen
Copy link

snaggen commented Mar 10, 2017

There is a library for fetching the preview information like this
https://github.com/LeonardoCardoso/Android-Link-Preview/

So when someone pastes/type in/shares a link, it shouldn't be that hard to use this to fetch the preview information. Then this could be sent as some kind of attachment with the link.

@Hooloovoo
Copy link

I would also love to see this.

Like others, I cannot see a privacy/security issue -- if I am sharing the link then I have visited it on my phone and there is no additional privacy risk by loading those links again through Signal (though Tor browsing would have to be managed).

As somebody else said, there is much more risk in me clicking an unexplained YouTube link that, in hindsight, I would rather not have viewed on work WiFi etc.

@FeuRenard
Copy link
Contributor

This feature in WhatsApp got some attention here:
https://twitter.com/mulander/status/874370124932943874

The WhatsApp app seems to attempt fetching a preview while you are typing a URL, like:

  • https://github.com/W
  • https://github.com/Wh
  • https://github.com/Whi
  • ...
  • https://github.com/WhisperSystems

Fetching is done on the sender's phone as suggested in this thread but the user agent seems to be exposed under certain conditions. If the user types the URL manually or pastes it from somewhere (not a browser) he/she didn't necessarily visit the URL (at least from the device he/she is using at the moment). If we decide to implement this, we should be careful.

@rdesimone
Copy link

IMO this feature is needed especially for users willing to switch from WhatsApp.

@Meteor0id
Copy link
Contributor

Meteor0id commented Mar 2, 2018

Additional idea:
When sharing a weblink, Twitter for example generates an image preview of the webpage, which automatically focuses on a certain place of the webpage. This place is usually the header text of the article or blogpost.

For signal I would propose to do a simulair thing, but with one addition: allow to user to edit the scroll position of the preview.

More in detail:

  1. when a weblink is placed in a message, genarate a screenshot of the entire page.
  2. look for the position of the header. To do this pharse the webpage for elements such as <main> <div id='main'> <div class="main"> <header> <article> <h1> and microdata attributes.
  3. cut the preview at the position which appears to be most relevant, and add this cut preview to the concept message
  4. add an edit button to this concept preview. The user may click this button, and the full page preview will be shown to him. He can than manually select on which area of the page he wishes the preview to be focussed.
  5. since this is a preview image of a webpage, there is no need for options to rotate the image or to draw on it. Citating certain areas or sentences can be done using Allow for the citation of web-pages, blog posts, and on-line articles. #7432 , and should not be done in preview images. Previews are meant to show a reliable image of what to expect when following the hyperlink.

These suggestions might be ambigious, but on long term these could be implemented (not nessesarily all at once). I just post them here now so the idea is open to discussion.

@automated-signal
Copy link

GitHub Issue Cleanup:
See #7598 for more information.

@signalapp signalapp locked and limited conversation to collaborators Apr 3, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
Development

No branches or pull requests