Skip to content

Use cboxexternal for access to publiclinks#5526

Merged
jessegeens merged 3 commits intorelease-3.7from
feat/cboxexternal-for-publiclink
Mar 31, 2026
Merged

Use cboxexternal for access to publiclinks#5526
jessegeens merged 3 commits intorelease-3.7from
feat/cboxexternal-for-publiclink

Conversation

@jessegeens
Copy link
Copy Markdown
Contributor

@jessegeens jessegeens commented Mar 3, 2026

No description provided.

@jessegeens jessegeens changed the title Feat/cboxexternal for publiclink Use cboxexternal for access to publiclinks Mar 4, 2026
@jessegeens jessegeens changed the base branch from master to refactor/eosfs-auth March 4, 2026 09:15
@jessegeens jessegeens force-pushed the refactor/eosfs-auth branch from 34d2c4f to e07a916 Compare March 4, 2026 09:16
@jessegeens jessegeens force-pushed the feat/cboxexternal-for-publiclink branch 7 times, most recently from afa323c to 3a6f2ee Compare March 4, 2026 09:37
@jessegeens jessegeens marked this pull request as ready for review March 4, 2026 09:38
@jessegeens jessegeens force-pushed the refactor/eosfs-auth branch from e07a916 to e5327fa Compare March 6, 2026 09:38
@jessegeens jessegeens changed the base branch from refactor/eosfs-auth to release-3.7 March 24, 2026 12:16
@jessegeens jessegeens force-pushed the feat/cboxexternal-for-publiclink branch from 3a6f2ee to 640cb2e Compare March 24, 2026 16:11
@jessegeens jessegeens requested a review from diocas March 24, 2026 16:12
@jessegeens jessegeens force-pushed the feat/cboxexternal-for-publiclink branch from 640cb2e to dfe5a53 Compare March 24, 2026 16:27
@jessegeens jessegeens force-pushed the feat/cboxexternal-for-publiclink branch 2 times, most recently from ab85ba2 to e11949a Compare March 25, 2026 13:05
@jessegeens jessegeens requested a review from diocas March 25, 2026 13:06
@jessegeens jessegeens force-pushed the release-3.7 branch 2 times, most recently from fc19a42 to f72abec Compare March 30, 2026 09:14
dependabot bot and others added 3 commits March 30, 2026 11:16
* Use a (configurable) dedicated service account for accesses made by external accounts,
  instead of impersonating the owner or using a token
* Renamed the different types of auth to be more clear (e.g. cboxAuth became systemAuth)
* Added a `InvalidAuthorization` to be returned instead of an empty auth; because empty auth maps to the system user (which is a sudo'er)
use instead a user of type `guest` for public link accesses,
with the id set to the publicshare id and access granted
through an added scope
@jessegeens jessegeens force-pushed the feat/cboxexternal-for-publiclink branch from e11949a to ffaed60 Compare March 30, 2026 09:17
@jessegeens jessegeens merged commit caa8bf4 into release-3.7 Mar 31, 2026
12 checks 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.

3 participants