Skip to content

Restrict Service Mode host file I/O and pam tunnel commands - #2289

Merged
amangalampalli-ks merged 4 commits into
fix/service-mode-attachment-and-pam-tunnelfrom
fix/service-mode-attachment-and-pam-tunnel-int
Aug 14, 2026
Merged

Restrict Service Mode host file I/O and pam tunnel commands#2289
amangalampalli-ks merged 4 commits into
fix/service-mode-attachment-and-pam-tunnelfrom
fix/service-mode-attachment-and-pam-tunnel-int

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 13, 2026
@amangalampalli-ks
amangalampalli-ks marked this pull request as ready for review August 13, 2026 11:14
@amangalampalli-ks
amangalampalli-ks merged commit c777bbe into fix/service-mode-attachment-and-pam-tunnel Aug 14, 2026
4 checks passed
amangalampalli-ks added a commit that referenced this pull request Aug 14, 2026
* 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 deleted the fix/service-mode-attachment-and-pam-tunnel-int branch August 14, 2026 17:48
@sk-keeper
sk-keeper restored the fix/service-mode-attachment-and-pam-tunnel-int branch August 14, 2026 17:48
sk-keeper pushed a commit that referenced this pull request Aug 14, 2026
* 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 pushed a commit that referenced this pull request Aug 14, 2026
* 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 deleted the fix/service-mode-attachment-and-pam-tunnel-int branch August 14, 2026 17:53
sk-keeper pushed a commit that referenced this pull request Aug 14, 2026
* 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
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.

3 participants