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

SUID workaround #256

Closed
wants to merge 3 commits into from
Closed

SUID workaround #256

wants to merge 3 commits into from

Conversation

jfgordon2
Copy link

Workaround for #222

Description

If the client exits upon start, it will automatically attempt to re-launch with the --no-sandbox tag as an attempt to address the SUID issue with Debian 10 and similar. This is a workaround to prefer automatically running the client instead of appearing to not work pending user intervention.

Screenshots

image

Release notes

Notes: no-notes

Reduce errors produced in terminal from debian package installations by testing for existence of symlink prior to executing unlink
Add a workaround to allow the client to run without user intervention on Debian 10 and similar when failing due to the SUID chrome-sandbox error
@shiftkey
Copy link
Owner

@jfgordon2 this is an interesting approach, but sadly is only visible when you're developing in the repository - these scripts are not part of what we distribute to end users when they install the app.

@jfgordon2 jfgordon2 closed this Apr 15, 2020
@jfgordon2
Copy link
Author

@shiftkey then back to the drawing board!

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.

2 participants