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

Various fix and improvements #197

Merged
merged 18 commits into from
Dec 13, 2021
Merged

Conversation

MagiusCHE
Copy link
Contributor

@MagiusCHE MagiusCHE commented Dec 13, 2021

I just started to work on various issues and i created this PR to prevent others from working on them.

Here the list of done ones.

affinity: 'main-window'
affinity: 'main-window',
contextIsolation: false,
nodeIntegration: true,
}
})
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

opt.semgrep.electron_context_isolation: Disabling context isolation can introduce Prototype Pollution vulnerabilities.
(at-me in a reply with help or ignore)

affinity: 'main-window'
affinity: 'main-window',
contextIsolation: false,
nodeIntegration: true,
}
})
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

opt.semgrep.electron_nodejs_integration: Node integration exposes node.js APIs to the electron app and this can introduce remote code execution vulnerabilities to the application if the app is vulnerable to Cross Site Scripting (XSS).
(at-me in a reply with help or ignore)

@julian-alarcon julian-alarcon merged commit a37d33a into julian-alarcon:main Dec 13, 2021
This was referenced Sep 19, 2023
julian-alarcon added a commit that referenced this pull request Nov 2, 2023
* Added "Show" to Tray menu to force show window and bring it to foreground (plasma is ignoring  tray "click" event).
* #193 parse --minimized via commandline to start app minimized; fix settings.getSync default
* Fixed trymenu broken setting. Note: hideOnMinimize doesn't work at the moment.
* Added minimize option on README

Co-authored-by: Julian Alarcon <alarconj@gmail.com>
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.

None yet

2 participants