Skip to content

feat(OCM): Implement ISignedCloudFederationProvider for deck - #8261

Merged
grnd-alt merged 1 commit into
mainfrom
kano-signed-fedration-provider
Aug 12, 2026
Merged

feat(OCM): Implement ISignedCloudFederationProvider for deck#8261
grnd-alt merged 1 commit into
mainfrom
kano-signed-fedration-provider

Conversation

@mickenordin

@mickenordin mickenordin commented Aug 12, 2026

Copy link
Copy Markdown
Contributor

Summary

When an app recieves OCM notifications, core needs to be able to resolve the remote server that sent the notification so that it can verify the signature of the notification.

This patch implements the ISignedCloudFederationProvider interface for the deck app, so that this can happen.

Checklist

When an app recieves OCM notifications, core needs to be able to resolve
the remote seerver that sent the notification so that it can verify the
signature of the notification.

This patch implements the ISignedCloudFederationProvider interface for
the deck app, so that this can happen.

Signed-off-by: Micke Nordin <kano@sunet.se>
@mickenordin
mickenordin force-pushed the kano-signed-fedration-provider branch from a6ac408 to f83d8ec Compare August 12, 2026 09:12
@mickenordin mickenordin changed the title feat(OCM): Implement ISigneCloudFederationProvider for deck feat(OCM): Implement ISignedCloudFederationProvider for deck Aug 12, 2026
@github-actions

Copy link
Copy Markdown
Contributor

馃悽 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 200191 was 198420 (+0.89%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

@github-actions

Copy link
Copy Markdown
Contributor

馃悽 Performance warning.
It looks like the query count of the integration tests increased with this PR.
Database query count is now 200172 was 198420 (+0.88%)
Please check your code again. If you added a new test this can be expected and the base value in tests/integration/base-query-count.txt can be increased.

@grnd-alt
grnd-alt merged commit 0d3857b into main Aug 12, 2026
40 of 41 checks passed
@grnd-alt
grnd-alt deleted the kano-signed-fedration-provider branch August 12, 2026 09:36
@github-actions

Copy link
Copy Markdown
Contributor

Hello there,
Thank you so much for taking the time and effort to create a pull request to our Nextcloud project.

We hope that the review process is going smooth and is helpful for you. We want to ensure your pull request is reviewed to your satisfaction. If you have a moment, our community management team would very much appreciate your feedback on your experience with this PR review process.

Your feedback is valuable to us as we continuously strive to improve our community developer experience. Please take a moment to complete our short survey by clicking on the following link: https://cloud.nextcloud.com/apps/forms/s/i9Ago4EQRZ7TWxjfmeEpPkf6

Thank you for contributing to Nextcloud and we hope to hear from you soon!

(If you believe you should not receive this message, you can add yourself to the blocklist.)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants