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

securely wrap execution #8451

Draft
wants to merge 6 commits into
base: develop
Choose a base branch
from
Draft

Conversation

ryan-willis
Copy link
Contributor

resolves INS-5048

what this does:

  • builds the normal Electron executable into Insomnia.dll (so that it can be code-signed)
  • builds a wrapper into Insomnia.exe (also code-signed)
  • when the wrapper runs, makes a copy of Insomnia.dll called insomnia-$VERSION.exe and executes it with custom Windows process mitigation policies
  • when Insomnia exits, the wrapper deletes the insomnia-$VERSION.exe file
  • standard output is piped directly

this is meant to be a stop-gap for CVE-2025-1353 and will very likely be superseded by other measures (NSIS/MSI installing into protected directories, etc)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant