Git Product home page Git Product logo

Comments (8)

ayumi-signal avatar ayumi-signal commented on July 18, 2024

Hello, sorry this is happening for you. I can confirm that CNBC links don't seem to generate previews. Debugging a little, it seems like the CNBC server just doesn't return content when the app makes the request. We can look into workarounds, but it suggests a remote server issue.

from signal-desktop.

deliriumcoc avatar deliriumcoc commented on July 18, 2024

It’s worth mentioning that CNBC link previews seem to work 100% of the time from Signal on my iPhone. This issue only happens on Desktop. So whatever the server is doing, it works fine with Signal mobile.

from signal-desktop.

deliriumcoc avatar deliriumcoc commented on July 18, 2024

@ayumi-signal Any update on this issue? Is it something the Signal team is looking at? It definitely seems like a desktop-only issue as I mentioned which is very curious. The iOS app does NOT have this issue.

from signal-desktop.

jamiebuilds-signal avatar jamiebuilds-signal commented on July 18, 2024

This is what we are getting back for that URL when we fetch it. It has an empty body, no title, no real content at all, from this response there's nothing we can really show.

Screenshot 2024-06-03 at 10 31 52 AM

I'm not exactly sure what this website is doing to give two different responses here, but there's nothing going wrong in the way we are fetching it, it's an issue with this website. So I'm going to close this issue.

from signal-desktop.

deliriumcoc avatar deliriumcoc commented on July 18, 2024

@jamiebuilds-signal Again, I think this is a real issue and I really wish you wouldn't just close this and write it off. CNBC worked fine for YEARS against the desktop client. Then it stopped working for a few months. Then it started working again. Now it's been broken for a few months. For this entire stretch of time the iOS Signal client works fine at generating previews of CNBC links. So clearly whatever CNBC returns to the iOS client is sufficient. It seems like there is some kind of difference between the desktop version and the iOS version.

from signal-desktop.

deliriumcoc avatar deliriumcoc commented on July 18, 2024

@jamiebuilds-signal An additional point I'd like to make here. This behavior has been off and on as mentioned over the past year. The desktop client at different points this past year would sometimes work and sometimes it wouldn't. As in there were times when I could link CNBC and the preview would show. Sometimes it wouldn't. The other ticket I mentioned in the OP I believes mentions some of this behavior. Sometimes you could post the link in desktop, it wouldn't show but you could delete the link, add a space, and repaste the link and it would work. It has just been flaky. The ios app version was rock solid throughout. There is some different behavior in the desktop client. There is a bug somewhere that needs to be found. Somehow it must be requesting the content differently from the mobile app. And as mentioned it has been intermittent at times in the past year. Sometimes intermittent in a single session. I even have seen instances where killing the desktop app and reloading it fixed it for a while. As a software developer myself it feels like there is an issue here.

from signal-desktop.

deliriumcoc avatar deliriumcoc commented on July 18, 2024

@jamiebuilds-signal @ayumi-signal

I noticed something worthwhile potentially for this issue. I'm hoping you guys will revisit. When I share a CNBC link on my iPhone to Signal it adds an additional __source to the URL. This seems to make a difference in the rendering of the preview. I tried using the same phone link on my DESKTOP Signal app and it works! That __source tag allows the Desktop Signal to render the preview. Do you have any idea if that __source is something that iOS Signal is adding? Or is this coming from the CNBC app perhaps?

https://www.cnbc.com/2024/06/18/ev-startup-fisker-files-for-bankruptcy-aims-to-sell-assets.html?__source=iosappshare%7Corg.whispersystems.signal.shareextension

image

from signal-desktop.

deliriumcoc avatar deliriumcoc commented on July 18, 2024

@jamiebuilds-signal @ayumi-signal Another followup to my discovery from last week. Pasting a link with the additional "__source=iosappshare%7Corg.whispersystems.signal.shareextension" extension seems to temporarily fix my Signal Desktop client. It's almost like it cached something for future requests and now it is getting good responses from cnbc.com that have previews. There is definitely something to this. I still urge you guys to investigate what is going on here.

from signal-desktop.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.