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

iOS: sharing crashes after a few seconds #1231

Closed
acdha opened this issue Aug 26, 2019 · 16 comments
Closed

iOS: sharing crashes after a few seconds #1231

acdha opened this issue Aug 26, 2019 · 16 comments

Comments

@acdha
Copy link

acdha commented Aug 26, 2019

If you start to share a page on nytimes.com, talkingpointsmemo.com or a few other sites the sharing dialog disappears after about 5 seconds. I’m not sure what those sites have running but it even happens in the Brave browser with JavaScript disabled.

@samuelclay
Copy link
Owner

I should admit that I never test in Brave. And I've noticed a lot of other inconsistencies in Brave for other sites that I'm working on.

I'm also assuming you mean the sharing bookmarklet? I'm working on that now, as I'm adding a save story (with saved story tags) to the bookmarklet, so I would be interested in figuring out what happens. Can you share a specific URL that is causing issues?

@acdha
Copy link
Author

acdha commented Aug 26, 2019

I'm using the iOS sharing panel in any app (Safari, Firefox, or Brave all reproduce). On the NYT homepage, it does not crash but if you pick any news story it disappears after a few seconds.

@acdha
Copy link
Author

acdha commented Dec 19, 2019

While sharing https://www.nytimes.com/interactive/2019/12/19/opinion/location-tracking-cell-phone.html I noticed that it also doesn't seem to crash on the interactives, which might be a clue what whatever the page is doing to cause the crash — I've been assuming that this is related to some sort of analytics or tracker script (which must not be in the blocklist for Firefox Focus) because all of the sites where I notice it on are media properties.

@samuelclay
Copy link
Owner

Yeah I'd like to get this fixed, we just launched it this year.

@Dejal
Copy link
Collaborator

Dejal commented Dec 21, 2019

To be clear, you are saying that when viewing a web page in the Brave browser, with the "Block Script" option enabled, then use the "Share with..." function and choose the NewsBlur sharing extension, it disappears after a few seconds?

I have tried to reproduce that (with New York Times, Talking Points Memo, and others, in both Brave and Safari), but the NewsBlur sharing extension has worked reliably.

Looking at the code, I can't see anything that could cause it to run out of memory or other issues (extensions are very memory constrained, so that seemed the most likely possibility).

Can either of you provide some examples of pages (and on which browsers) that exhibit the issue? A screen recording showing it could help, too.

@acdha
Copy link
Author

acdha commented Dec 21, 2019

In Brave, Firefox, or Chrome it crashes almost immediately. In Safari it usually crashes after a second. For example, just now I opened Safari and clicked on the first story on nytimes.com (https://www.nytimes.com/2019/12/21/world/asia/north-korea-missile-test-trump-kim.html) and used the sharing extension. It loaded and disappeared about a second later.

@acdha
Copy link
Author

acdha commented Dec 21, 2019

The second time, following the link directly, it did not crash in Safari.

@samuelclay
Copy link
Owner

Can you provide a screenshot of right before it crashes?

@acdha
Copy link
Author

acdha commented Dec 24, 2019

I can’t get a screenshot before it crashes unless you mean before I open the sharing extension at all: it closes quickly.

I did notice it failed in the iOS app trying to share a story from someone else’s blurblog (https://mareino.newsblur.com/).

@Dejal
Copy link
Collaborator

Dejal commented Dec 24, 2019

A screen recording could be helpful.

@acdha
Copy link
Author

acdha commented Dec 28, 2019

@github prevents attacking these directly but the behavior is identical in both Safari and Firefox, latest versions of everything:

https://www.dropbox.com/s/bglq962v3gd33ci/RPReplay_Final1577494511.MP4?dl=0
https://www.dropbox.com/s/rxer0op8hqyxmv7/RPReplay_Final1577494571.MP4?dl=0

@acdha
Copy link
Author

acdha commented Jan 12, 2020

@Dejal
Copy link
Collaborator

Dejal commented Jan 25, 2020

Thanks. Does it crash every time with that Bloomberg article, or sporadically? When I tried it, it crashed once, but I couldn't get it to happen again, despite trying for some time. This is the most frustrating kind of issue; almost impossible to reproduce.

Looking at the code, I can't see any obvious points of failure. The share extension only gets called twice before displaying the share panel; once to ask if the item applies, which doesn't do anything complex, and again to optionally add extra UI, which doesn't do anything at all.

The most likely cause is running out of memory, but the NewsBlur share extension doesn't allocate anything in those calls. So if that is the cause, it's at the OS level, which seems strange.

Another thing to try: go to a page that fails like the above, and try accessing other share extensions (e.g. if you use it, Slack uses a very basic share extension like NewsBlur's). Do they work?

@acdha
Copy link
Author

acdha commented Jan 25, 2020 via email

@samuelclay
Copy link
Owner

Ok I just tried to share a story from the NYT app and it shared a blank story with no content (but the permalink worked. The title morphed from the actual story title "Gone from Grocery Shelves, Now There's a Mad Dash to Find Them" to "The Coronavirus Outbreak" which is the series title. I do blame the NYT app, but when I send to Mail, I get the right title.

@acdha
Copy link
Author

acdha commented May 11, 2020

I've noticed that happening in the browser, too — the share works but only includes the title and URL while other apps see the entire page.

Dejal added a commit that referenced this issue May 23, 2020
- Now extracts the URL title and sets that when sharing, in addition to any user comments.
@Dejal Dejal closed this as completed May 23, 2020
samuelclay pushed a commit that referenced this issue Jun 16, 2020
- Now extracts the URL title and sets that when sharing, in addition to any user comments.
samuelclay added a commit that referenced this issue Jun 16, 2020
* master:
  Android v10.0b3.
  #1298 (saved searches)
  #1298 (saved searches)
  #1304 (not restoring state)
  #1306 (status bar during fullscreen)
  #1231 (sharing crash)
  #1309 (theme in story detail view)
  Fixed #1314 (persistent "All Done")
  #1306 (status bar during fullscreen)
  #1294 (move to new folder)
  #1293 (invalid permalink)
  #1306 (status bar during fullscreen)
  #1306 (status bar during fullscreen)
  #1294 (move to new folder)
  #1301 (white flash in dark mode)
  #1270 (replacing UIWebView with WKWebView)
  #1270 (replacing UIWebView with WKWebView)
  #1270 (replacing UIWebView with WKWebView)
  #1270 (replacing UIWebView with WKWebView)
  #1270 (replacing UIWebView with WKWebView)
samuelclay added a commit that referenced this issue Jun 22, 2020
* master: (63 commits)
  Better newsletter check
  Adding allowlist (whitelist) for RSS Bridge.
  Removing last vestigates of Google Analytics.
  Android v10.0.
  #1161 (android widget) RemoteViews empty state
  Android v10.0b3.
  #1298 (saved searches)
  #1298 (saved searches)
  #1304 (not restoring state)
  #1306 (status bar during fullscreen)
  #1231 (sharing crash)
  #1309 (theme in story detail view)
  Fixed #1314 (persistent "All Done")
  #1306 (status bar during fullscreen)
  #1294 (move to new folder)
  #1293 (invalid permalink)
  #1306 (status bar during fullscreen)
  #1306 (status bar during fullscreen)
  #1294 (move to new folder)
  #1301 (white flash in dark mode)
  ...
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

3 participants