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

[Crash] when tapping on a repost #1669

Open
mplorentz opened this issue Oct 18, 2024 · 3 comments
Open

[Crash] when tapping on a repost #1669

mplorentz opened this issue Oct 18, 2024 · 3 comments
Assignees
Labels
bug Something isn't working

Comments

@mplorentz
Copy link
Member

mplorentz commented Oct 18, 2024

Describe the bug

A crash in Router.swift:67 when tapping on a repost.

To Reproduce

None yet

Screenshots or Video

If applicable, add screenshots or video to help explain your problem.

Technical info

  • OS: iOS
  • App Version: 0.2.3 (501 and 504)
  • Environment: Staging

Additional context

Slack report one (with logs)
testflight crash log one
Slack report two (with logs)
testflight crash log two
Sentry issue

The crash is on the line fatalError("Tried to push a note with no identifier; that's not going to work."). Probably this should just be an assertion failure? But we still need to fix whatever code is creating a note with no identifier.

It's weird that the crashes reported in slack and testflight don't match up with the ones in Sentry.

@mplorentz mplorentz added the bug Something isn't working label Oct 18, 2024
@mplorentz mplorentz moved this to Sprint in Nos Product Board Oct 21, 2024
@bryanmontz
Copy link
Contributor

This should definitely be assertionFailure(_:) instead, so that it only crashes in dev and not production.

@pelumy
Copy link
Contributor

pelumy commented Oct 22, 2024

Thanks for the tip @bryanmontz.

@mplorentz
Copy link
Member Author

I just saw this crash when I tapped on the note quoted in note10x5wxcr37lzgrmqfwkq3dtzp5vwkl6qhre27736vuncre5prjerq9vpdp0. The quoted note hadn't finished loading when I tapped it, it was still spinning.

The logs showed that the pushed note had no author, identifier, or replaceableIdentifier.

I relaunched the app to see if I could reproduce but I couldn't. Maybe if I deleted the app I could but I don't have time for that right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
Status: Code Review
Development

No branches or pull requests

3 participants