Skip to content
This repository has been archived by the owner on Jan 31, 2024. It is now read-only.

restrict osascript for frontend #355

Merged
merged 1 commit into from
Oct 4, 2022
Merged

restrict osascript for frontend #355

merged 1 commit into from
Oct 4, 2022

Conversation

ozgunozerk
Copy link
Contributor

  1. we have a new long regex... This will help to prevent commands other than those defined by us to be executed. Stepping towards fixing the frontend privilege issue Trail of Bits audit - Issue #5 (Insufficient privilege separation between the frontend and backend) #339
  2. disableAutoLaunch was creating a bug (where user disables the autolaunch, and restarts the application), so I removed that logic completely. I figured out it was not helpful to the user in any of the possible scenarios.
  3. prettierrc compliance with eslint

@ozgunozerk ozgunozerk added this to the Audit Fixes milestone Oct 4, 2022
@ozgunozerk ozgunozerk merged commit 704f5fc into main Oct 4, 2022
@ozgunozerk ozgunozerk deleted the restrict-shell branch October 4, 2022 16:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

Trail of Bits audit - Issue #5 (Insufficient privilege separation between the frontend and backend)
3 participants