feat(federation): Federated table sharing by benjaminfrueh · Pull Request #2786 · nextcloud/tables

benjaminfrueh · 2026-07-03T14:38:59Z

Federated table sharing

Allows sharing tables with users on other Nextcloud instances.

Todo

🏁 Checklist

⛑️ Tests (unit and/or integration) are included or not needed
🔙 Backport requests are created or not needed: /backport to stableX.X
📅 Milestone is set
🌸 PR title is meaningful (if it should be in the changelog: is it meaningful to users?)

🤖 AI (if applicable)

The content of this PR was partly or fully generated using AI

benjaminfrueh · 2026-07-17T08:07:33Z

Did not entirely check all code paths yet. But the general structure looks good 🔥

What I noticed though: when creating a federated share it instantly shows as a new table on the receiving user's view. In Deck I think we're sending a notification that the user has to confirm before it shows in the app, so user's don't get spammed.

Thanks for the feedback! After checking Deck's shareReceived(), it also seems to insert the board immediately and just sends an informational notification without accept/decline flow.
An "Pending shares" acceptance flow, similar to how Files handles federated shares would maybe be a good addition to both deck and tables federated sharing - maybe we can consider it as a follow-up feature?

blizzz · 2026-07-22T18:39:31Z

 namespace OCA\Tables\AppInfo;

 use Exception;
+use OC\OCM\OCMSignatoryManager;


Would be cool (and more sustainable) when there will be a public interface for this, but definitely out of scope for now, might be accompanied by an issue against server if there is not one yet.

(On second look it might be more complicated than that)

blizzz

some things that would be good to have, but no definite blocker by reading the code

blizzz · 2026-07-22T18:55:25Z

+	#[RequirePermission(permission: Application::PERMISSION_UPDATE, typeParam: 'nodeCollection')]
+	#[ApiRoute(verb: 'PUT', url: '/api/2/{nodeCollection}/{nodeId}/rows/{rowId}', requirements: ['nodeCollection' => '(tables|views)', 'nodeId' => '(\d+)'])]
+	public function updateRow(string $nodeCollection, int $nodeId, int $rowId, mixed $data): DataResponse {
+		if (is_string($data)) {


$data could be of type string|array then?

blizzz · 2026-07-22T19:53:46Z

+	}
+
+	public function notifyPermissionUpdate(Share $share): void {
+		$this->proxy->sendNotification(


what if any of those requests do not get through (maybe the remote nextcloud is being updated), will they be retried?

Currently no retry mechanism, if the notification fails it is logged as a warning.

blizzz · 2026-07-22T20:01:45Z

 				<referencedClass name="Doctrine\DBAL\Types\Types" />
 				<referencedClass name="OC\Core\Command\Base" />
+				<referencedClass name="OC\Http\Client\Response" />
+				<referencedClass name="OC\OCM\OCMSignatoryManager" />


stub would be preferred

blizzz · 2026-07-22T20:02:04Z

 				<referencedClass name="Doctrine\DBAL\Schema\Table" />
 				<referencedClass name="Doctrine\DBAL\Types\Types" />
 				<referencedClass name="OC\Core\Command\Base" />
+				<referencedClass name="OC\Http\Client\Response" />


this should have an interface?

Signed-off-by: Benjamin Frueh <benjamin.frueh@gmail.com> # Conflicts: # lib/AppInfo/Application.php # psalm.xml

Signed-off-by: samin-z <samin.zavarkesh@gmail.com>

Signed-off-by: Benjamin Frueh <benjamin.frueh@gmail.com>

… signature verification Signed-off-by: Benjamin Frueh <benjamin.frueh@gmail.com>

blizzz · 2026-08-24T14:42:34Z

[Follow up/General tech debt] One thing about Migrations, because i just stumbled across them recently, are the attributes. Can be added as followup, and I do not think we have them in place elsewhere. https://docs.nextcloud.com/server/latest/developer_manual/basics/storage/migrations.html#migrations-and-metadata

benjaminfrueh requested review from blizzz and enjeck as code owners July 3, 2026 14:39

benjaminfrueh marked this pull request as draft July 3, 2026 14:40

benjaminfrueh requested a review from grnd-alt July 3, 2026 14:41

benjaminfrueh changed the title ~~feat(federation): schema, outgoing share and frontend support~~ feat(federation): Federated Table Sharing Jul 3, 2026

benjaminfrueh changed the title ~~feat(federation): Federated Table Sharing~~ feat(federation): Federated table sharing Jul 3, 2026

AndyScherzinger added enhancement New feature or request 2. developing Work in progress labels Jul 3, 2026

benjaminfrueh added this to 📝 Productivity team Jul 3, 2026

github-project-automation Bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Productivity team Jul 3, 2026

benjaminfrueh moved this from 🧭 Planning evaluation (don't pick) to 🏗️ In progress in 📝 Productivity team Jul 3, 2026

benjaminfrueh force-pushed the feat/federated-sharing branch 7 times, most recently from 403289e to ba97e23 Compare July 8, 2026 07:13

grnd-alt reviewed Jul 9, 2026

View reviewed changes

Comment thread lib/Controller/FederationColumnOCSController.php Outdated

benjaminfrueh force-pushed the feat/federated-sharing branch from 4da0386 to 35b4e5e Compare July 13, 2026 16:28

benjaminfrueh marked this pull request as ready for review July 14, 2026 14:33

benjaminfrueh force-pushed the feat/federated-sharing branch 2 times, most recently from 1bf2714 to e948b30 Compare July 14, 2026 14:46

benjaminfrueh added this to the v2.3.0 milestone Jul 14, 2026

benjaminfrueh force-pushed the feat/federated-sharing branch 5 times, most recently from 8a02508 to 63c7628 Compare July 16, 2026 08:50

nextcloud-command added the AI assisted label Jul 16, 2026

benjaminfrueh force-pushed the feat/federated-sharing branch from 4beef8f to a38af3d Compare July 17, 2026 14:10

benjaminfrueh requested a review from grnd-alt July 17, 2026 14:11

blizzz force-pushed the feat/federated-sharing branch from a38af3d to e6485cc Compare July 22, 2026 18:30

blizzz reviewed Jul 22, 2026

View reviewed changes

Comment thread lib/Federation/FederationProxy.php

blizzz reviewed Jul 22, 2026

View reviewed changes

Comment thread lib/Federation/FederationProxy.php Outdated

blizzz requested changes Jul 22, 2026

View reviewed changes

Comment thread lib/Controller/Api1Controller.php Outdated

samin-z force-pushed the feat/federated-sharing branch 2 times, most recently from ac23763 to 6e4b66d Compare August 6, 2026 15:14

benjaminfrueh and others added 5 commits August 20, 2026 20:18

feat(federation): federated table and view sharing

7438480

Signed-off-by: Benjamin Frueh <benjamin.frueh@gmail.com> # Conflicts: # lib/AppInfo/Application.php # psalm.xml

fix: when federation is disabled, local table APIs dont fail

00865d6

Signed-off-by: samin-z <samin.zavarkesh@gmail.com>

fix: remove duplicate federation enabled checks

f8e6900

Signed-off-by: Benjamin Frueh <benjamin.frueh@gmail.com>

fix: validate nodeType also on receiver side

4fb4b1f

Signed-off-by: Benjamin Frueh <benjamin.frueh@gmail.com>

fix: add declare strict to new files

55bae93

Signed-off-by: Benjamin Frueh <benjamin.frueh@gmail.com>

benjaminfrueh force-pushed the feat/federated-sharing branch from 3bad71c to aec831b Compare August 20, 2026 18:21

fix: throw FederationRequestError instead of generic Exception

eb8eb93

Signed-off-by: Benjamin Frueh <benjamin.frueh@gmail.com>

benjaminfrueh force-pushed the feat/federated-sharing branch from aec831b to eb8eb93 Compare August 20, 2026 18:27

chore: update migration

63f67ab

Signed-off-by: Benjamin Frueh <benjamin.frueh@gmail.com>

benjaminfrueh force-pushed the feat/federated-sharing branch from aa3787c to 198e5a2 Compare August 21, 2026 18:52

fix: send node uuid to receiving instance

c0f2f78

Signed-off-by: Benjamin Frueh <benjamin.frueh@gmail.com>

benjaminfrueh force-pushed the feat/federated-sharing branch from 198e5a2 to c0f2f78 Compare August 21, 2026 18:52

fix: implement ISignedCloudFederationProvider to fix OCM notification…

521099a

… signature verification Signed-off-by: Benjamin Frueh <benjamin.frueh@gmail.com>

blizzz approved these changes Aug 24, 2026

View reviewed changes

Merge branch 'main' into feat/federated-sharing

3693248

blizzz merged commit ded3ca0 into main Aug 27, 2026
79 checks passed

blizzz deleted the feat/federated-sharing branch August 27, 2026 07:48

github-project-automation Bot moved this from 🏗️ In progress to ☑️ Done in 📝 Productivity team Aug 27, 2026

Uh oh!

Conversation

benjaminfrueh commented Jul 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Federated table sharing

Todo

🏁 Checklist

🤖 AI (if applicable)

Uh oh!

Uh oh!

benjaminfrueh commented Jul 17, 2026

Uh oh!

blizzz Jul 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

blizzz left a comment

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

blizzz Jul 22, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

blizzz Jul 22, 2026

Choose a reason for hiding this comment

Uh oh!

benjaminfrueh Aug 20, 2026

Choose a reason for hiding this comment

Uh oh!

blizzz Jul 22, 2026

Choose a reason for hiding this comment

Uh oh!

blizzz Jul 22, 2026

Choose a reason for hiding this comment

Uh oh!

Uh oh!

Uh oh!

Uh oh!

blizzz commented Aug 24, 2026

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

6 participants

benjaminfrueh commented Jul 3, 2026 •

edited

Loading

blizzz Jul 22, 2026 •

edited

Loading