-
Notifications
You must be signed in to change notification settings - Fork 67
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
Fix warning message on Mac about secure coding not enabled #632
base: master
Are you sure you want to change the base?
Conversation
- See eclipse-equinox#630 - On macOS 14 and later a warning message is written to console: "WARNING: Secure coding is not enabled for restorable state! Enable secure coding by implementing NSApplicationDelegate.applicationSupportsSecureRestorableState: and returning YES." - This adds an NSApplicationDelegate that implements applicationSupportsSecureRestorableState
011c61d
to
d15cb06
Compare
@sravanlakkimsetti can you review this? @Phillipus thank you for being pro-active on this topic. You can try out the executables/launcher-libraries archived in the Jenkins verification build to test if it has the desired effect. |
@HannesWell Thanks for the heads up. I had already compiled and built the |
I'm not certain that setting an app delegate where I have set it in the PR is the right place. There are places that reference a an existing delegate: equinox/features/org.eclipse.equinox.executable.feature/library/cocoa/eclipseCocoa.c Lines 197 to 198 in c4be7f2
equinox/features/org.eclipse.equinox.executable.feature/library/cocoa/eclipseCocoa.c Lines 211 to 212 in c4be7f2
equinox/features/org.eclipse.equinox.executable.feature/library/cocoa/eclipseCocoa.c Lines 234 to 235 in c4be7f2
I'm not sure if setting a new delegate where I have might over-write a possibly existing one and then those functions might not work. |
I did a check, and at the point where this PR sets the AppDelegate it is |
@subyssurendran666 Can you please take a look? Lets merge this after 4.32 |
Don't merge this one without further investigation. The equivalent implementation in SWT caused a crash to desktop. See eclipse-platform/eclipse.platform.swt#1240 |
See [Mac Sonoma] "WARNING: Secure coding is not enabled for restorable state" warning message #630
On macOS 14 and later a warning message is written to console: "WARNING: Secure coding is not enabled for restorable state! Enable secure coding by implementing NSApplicationDelegate.applicationSupportsSecureRestorableState: and returning YES."
This adds an NSApplicationDelegate that implements applicationSupportsSecureRestorableState
I don't know if the additional code could be better placed. Feedback welcome!