-
Notifications
You must be signed in to change notification settings - Fork 55
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
Support for sudo with tty-tickets enabled (MacOS Sierra by default) #39
Comments
tty-tickets
enabled (MacOS Sierra by default)
Hello! Just want to check if this is the issue i'm hitting. I have a simple app just trying to test out electron-sudo. Here is my code and the output. Running on MacOS Sierra. var Sudoer = require('electron-sudo').default;
let options = {name: 'electron sudo application'};
var sudoer = new Sudoer(options);
sudoer.spawn('ls', ['/']).then(function (cp) {
cp.stdout.on('data', (msg) => {
console.log('Looks like we have a message on STDOUT');
console.log(err.toString('utf8'));
});
cp.stderr.on('data',(err) => {
console.log('Looks like we have a message on STDERR');
console.log(err.toString('utf8'));
});
cp.on('close',() => {
console.log('Processed Finished!');
});
});
|
@automation-stack Hi Aleksandr. Do you have any ideas how to fix that? |
@igor-lemon yes, it will be fixed soon, |
Cool. Thank you. I'm looking forward changes. :) |
@automation-stack So is this completely broken on MacOS Sierra or can it still be used? Do you have an ETA on the fix? |
@justechn working hard on this issues, details will follow |
Any chance this is fixed? I have been using sudo-prompt but it is actually quite poor in that it doesn't give you access to the underlying child process after it is created. |
Same issue here. macOS Sierra 10.12.5. |
Would also love to see this working again. If there's anything we can do to help, let us know! |
@DSBalaban the idea is to refuse to use |
I ran into the same issue and made it work on Sierra at least by using The change is hacky but might help someone running into this problem, mullvad@f71134f |
@eriklarko i think you should post a PR to @automation-stack :) |
Personally I feel the |
@imdreamrunner yeah but how's that any different from using sudo and what difference does it make if electron-sudo is broken for over 6 months.. |
Is electron-sudo still maintaining? any updates for this issue? |
https://derflounder.wordpress.com/2016/09/21/tty_tickets-option-now-on-by-default-for-macos-sierras-sudo-tool/
The text was updated successfully, but these errors were encountered: