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

Installed Electron App crashes after upgrading to 2.3.0 #449

Closed
TrofinSorin opened this issue Nov 6, 2022 · 5 comments
Closed

Installed Electron App crashes after upgrading to 2.3.0 #449

TrofinSorin opened this issue Nov 6, 2022 · 5 comments
Labels
DONE Implementation is done, but not yet merged into master

Comments

@TrofinSorin
Copy link

Version
NutJS 2.3.0
Node 14.12.0
MacOS Catalina 10.15.7

Short overview
After upgrading NutJS from 2.2.1 to 2.3.0 the packaged and installed electron app crashes after running it.

Issue occurs on

MacOS Catalina 10.15.7

Detailed error description
NutJS 2 3 0 Error

Steps to reproduce error

  1. Upgrade to 2.3.0
  2. Package Electron app for MacOS
  3. Install the app
  4. Run the app and you will get an error and it will crash
@s1hofmann
Copy link
Member

Hi @TrofinSorin 👋

That's interesting. The app crashes due to a failing symbol lookup for CGPreflightScreenCaptureAccess, which, if you follow the link to the Apple developer docs, is supposed to be available in macOS 10.15+. (Was introduced in nut.js 2.3.0 to check required permissions on startup)

Since you're running 10.15.7, this shouldn't be a problem.

Do you have a hard requirement for this version of macOS? Or could you double check with a later version?

Best regards

Simon

@TrofinSorin
Copy link
Author

TrofinSorin commented Nov 6, 2022

Hello @s1hofmann, thank you for your fast answer.

I think my MacOS cannot go higher than 10.15.7 for now, it's an older MacOS.

Maybe it can be checked if the symbol exists and if does not exist instead of the crash maybe it can behave the same as 2.2.1 with no permissions checking on startup.

@s1hofmann
Copy link
Member

I'll see to find a sustainable solution

@s1hofmann
Copy link
Member

Hey @TrofinSorin 👋

Could you give @nut-tree/nut-js@next a try?

@TrofinSorin
Copy link
Author

@s1hofmann it works on @nut-tree/nut-js@next !!

@s1hofmann s1hofmann added the DONE Implementation is done, but not yet merged into master label Nov 14, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
DONE Implementation is done, but not yet merged into master
Projects
None yet
Development

No branches or pull requests

2 participants