emit audit events when SFTP or SCP commands are blocked#36977
emit audit events when SFTP or SCP commands are blocked#36977capnspacehook merged 5 commits intomasterfrom
Conversation
capnspacehook
added
server-access
sftp
github-actions
Bot
added
audit-log
|
The PR changelog entry failed validation: Changelog entry not found in the PR body. Please add a "no-changelog" label to the PR, or changelog lines starting with |
| type: 'sftp', | ||
| desc: 'SFTP Disallowed', | ||
| format: ({ user, path, ...rest }) => | ||
| `User [${user}] was blocked from creating a SFTP session on node [${ |
There was a problem hiding this comment.
| `User [${user}] was blocked from creating a SFTP session on node [${ | |
| `User [${user}] was blocked from creating an SFTP session on node [${ |
There was a problem hiding this comment.
Wouldn't it be 'a' since SFTP starts with a consonant, 'S'?
There was a problem hiding this comment.
In case of acronyms, I was taught to use a/an depending on whether the way you pronounce the acronym starts with a consonant or a vowel sound.
https://www.scribbr.com/commonly-confused-words/a-vs-an/#acronyms
An MRI [em-arr-eye] machine uses magnets to take detailed scans of your organs.
It can be expensive to pay for an SAT [ess-ay-tee] prep course out of pocket.
|
@capnspacehook let's address Rafal's grammar suggestion and get this merged. V15 is coming soon! |
zmb3
left a comment
zmb3
left a comment
There was a problem hiding this comment.
LGTM once you fix the unused import
github-merge-queue
Bot
removed this pull request from the merge queue due to failed status checks
|
@capnspacehook See the table below for backport results.
|
4 participants
Fixes #36820.
changelog: emit audit events when SFTP or SCP commands are blocked