Skip to content

KC-1410, KC-1413: Restrict Service Mode host file I/O and pam tunnel commands - #2290

Merged
sk-keeper merged 1 commit into
releasefrom
fix/service-mode-attachment-and-pam-tunnel
Aug 14, 2026
Merged

KC-1410, KC-1413: Restrict Service Mode host file I/O and pam tunnel commands#2290
sk-keeper merged 1 commit into
releasefrom
fix/service-mode-attachment-and-pam-tunnel

Conversation

@amangalampalli-ks

Copy link
Copy Markdown
Contributor

Summary

  • Block local filesystem attachment uploads/downloads in Service Mode so a limited API key cannot read or write host files via record-add, record-update, or download-attachment.
  • Block pam tunnel subcommands other than edit in Service Mode to close remote shell execution via pam tunnel start --run.

Changes

  • Reject local-path file attachments in Service Mode for record-add / record-update.
  • Reject download-attachment in Service Mode.
  • Allow only pam tunnel edit through Service Mode policy checks; block start/list/stop/diagnose (including --run).
  • Add unit tests for the attachment and auth/policy guards.

@amangalampalli-ks amangalampalli-ks self-assigned this Aug 14, 2026
@amangalampalli-ks
amangalampalli-ks force-pushed the fix/service-mode-attachment-and-pam-tunnel branch from c777bbe to 1b506ba Compare August 14, 2026 13:19
@amangalampalli-ks
amangalampalli-ks marked this pull request as ready for review August 14, 2026 13:25
* Restrict Service Mode host file I/O and pam tunnel commands

* Fix review comments

* Place all service mode restrictions at one place

* Add ra ru in restricted list
@sk-keeper
sk-keeper force-pushed the fix/service-mode-attachment-and-pam-tunnel branch from 1b506ba to 359b1d3 Compare August 14, 2026 17:52
@sk-keeper
sk-keeper merged commit 1eeeaa0 into release Aug 14, 2026
4 checks passed
@sk-keeper
sk-keeper deleted the fix/service-mode-attachment-and-pam-tunnel branch August 14, 2026 17:53
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