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

Application hang when calling panic #904

Closed
timmattison opened this issue Jan 17, 2024 · 2 comments
Closed

Application hang when calling panic #904

timmattison opened this issue Jan 17, 2024 · 2 comments
Labels
duplicate This issue or pull request already exists

Comments

@timmattison
Copy link
Contributor

Describe the bug
I have gotten my application to hang in a way that I can't get out of it without killing it from another terminal when calling panic while the application is initializing. If the application panics at a later time it exits properly.

Setup
Please complete the following information along with version numbers, if applicable.

  • OS - macOS Sonoma 14.2.1
  • Shell - zsh
  • Terminal Emulator - iterm2
  • Terminal Multiplexer - N/A

To Reproduce
Steps to reproduce the behavior:

  1. Clone https://github.com/timmattison/bubbletea-hang
  2. Run go run main2.go model.go
  3. Kill the process from another terminal

Source Code
Source code is in linked repo

Expected behavior
I would expect that it panics and exits but instead gets stuck when trying to write to the done channel.

r.done <- struct{}{}

@timmattison
Copy link
Contributor Author

I think #905 should fix this but I haven't had enough time to test it. Can anyone take a look and see what you think?

@muesli
Copy link
Contributor

muesli commented Mar 1, 2024

Closing as dupe of #924.

@muesli muesli closed this as completed Mar 1, 2024
@muesli muesli added the duplicate This issue or pull request already exists label Mar 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
duplicate This issue or pull request already exists
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants