Skip to content

server: introduce USE_RECIPIENT for reading other users' recipients - #119

Merged
littlegrasscao merged 2 commits into
mainfrom
server/split2/11c-use-recipient
Oct 21, 2026
Merged

littlegrasscao merged 2 commits into
mainfrom
server/split2/11c-use-recipient

Conversation

@littlegrasscao

@littlegrasscao littlegrasscao commented Oct 15, 2026 •

Copy link
Copy Markdown
Collaborator

Summary

  • add the USE_RECIPIENT privilege, checked through CatalogConnector#authorize
  • GET /recipients lists every recipient for a caller with USE_RECIPIENT, and only the caller's own recipients otherwise
  • GET /recipients/{recipient} needs the caller to own the recipient or hold USE_RECIPIENT; otherwise it returns 403 PERMISSION_DENIED. Callers can no longer see the activation URL of someone else's recipient without the privilege
  • alice and bob in the sample local-catalog.yml also hold USE_RECIPIENT; carol still does not
  • replaces the USE_RECIPIENT TODOs from server: add provider-admin recipient entities and CRUD #98

Testing

  • mvn clean install: 125 tests passed
  • RecipientReadPrivilegeTest:
    • carol, without USE_RECIPIENT, lists only her own recipient, while bob, who holds USE_RECIPIENT, lists both alice's and carol's
    • carol can get her own recipient but gets 403 for alice's; bob can read carol's

@littlegrasscao
littlegrasscao force-pushed the server/split2/11c-use-recipient branch from 286e826 to 4fdf396 Compare October 16, 2026 00:17
@littlegrasscao
littlegrasscao force-pushed the server/split2/11c-use-recipient branch from 4fdf396 to 1f67e34 Compare October 16, 2026 00:41
@littlegrasscao
littlegrasscao force-pushed the server/split2/11c-use-recipient branch from 1f67e34 to 4d4e335 Compare October 19, 2026 17:12
@zhu-tom
zhu-tom added this pull request to stack #116 October 19, 2026 19:06
@littlegrasscao
littlegrasscao force-pushed the server/split2/11c-use-recipient branch from 4d4e335 to 61794b9 Compare October 19, 2026 21:43
@littlegrasscao
littlegrasscao force-pushed the server/split2/11c-use-recipient branch 3 times, most recently from a251778 to a729461 Compare October 19, 2026 22:31
@littlegrasscao
littlegrasscao force-pushed the server/split2/11c-use-recipient branch 2 times, most recently from b9bc3fd to ea28c0b Compare October 21, 2026 17:01
Base automatically changed from server/split2/11b-use-share to main October 21, 2026 18:54
@littlegrasscao
littlegrasscao force-pushed the server/split2/11c-use-recipient branch from ea28c0b to 09b066d Compare October 21, 2026 18:55
@littlegrasscao
littlegrasscao merged commit baf4398 into main Oct 21, 2026
1 check passed
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