3 ms·
It's only removed if sudo doesn't send mail. It still warns if it's relevant.
by stefncb 3y ago
It's only removed if sudo doesn't send mail. It still warns if it's relevant.
- electroly 3y agoThis is a pretty short diff and it clearly does NOT do that, unless you're saying they went back later in another commit to add this. They removed the message in all situations. The string "This incident will be reported" has been removed from the source code; it could not possibly print that message now.
- simse 3y agoThe warning is added back here: https://github.com/sudo-project/sudo/commit/9757d29a24ac1872872cf09757b0439c54089707 https://github.com/sudo-project/sudo/commit/9757d29a24ac1872...
- fafqg 3y agoThis doesn't seem to be correct.
- freedomben 3y agoIs there more to the change than the linked commit? Because if not, looking at the code change I don't see how you could possibly be correct. There is no additional logic/branching that could be checking whether sudo sends mail or not, just a string change.
- TaylorAlexander 3y agoIt was added back in a later commit but only prints if the mail is configured.
- freedomben 3y agoNice, thanks that's very useful info
- bo0tzz 3y agoThis is correct, and was added back a few weeks later in https://github.com/sudo-project/sudo/commit/9757d29a24ac1872872cf09757b0439c54089707 https://github.com/sudo-project/sudo/commit/9757d29a24ac1872...
- can16358p 3y agoIsn't it leaking detail about internal policy about whether incorrect sudoing is getting reported or not though?