Skip to content

Add AppPortal reverse proxy access control helpers#378

Open
jmcte wants to merge 1 commit intoN4S4:masterfrom
OMT-Global:codex/issue-179-reverse-proxy-access-control
Open

Add AppPortal reverse proxy access control helpers#378
jmcte wants to merge 1 commit intoN4S4:masterfrom
OMT-Global:codex/issue-179-reverse-proxy-access-control

Conversation

@jmcte
Copy link
Copy Markdown
Contributor

@jmcte jmcte commented Apr 26, 2026

Summary

  • add AppPortal AccessControl list/update helpers with JSON entry serialization
  • fix ReverseProxy list handling and add create/update/delete helpers
  • add request-contract coverage for AccessControl and ReverseProxy payloads

Validation

  • python -m pytest tests/test_core_service_apps.py -q
  • PYTHONPATH=tests python -m pytest -q --ignore=tests/integration --ignore=tests/test_add_s3_sync_task.py --ignore=tests/test_core_user.py --ignore=tests/test_syno_api.py
  • python -m compileall -q synology_api/core_service_apps.py tests/test_core_service_apps.py
  • git diff --check
  • live DSM add/remove smoke test for a temporary reverse proxy entry; final temp count returned to 0

Closes #179

@jmcte jmcte force-pushed the codex/issue-179-reverse-proxy-access-control branch from 88151cb to 26086c3 Compare April 26, 2026 21:36
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.

Add reverse proxy settings function

1 participant