Skip to content

fix: do not fetch remote collabora url for own server#4714

Merged
elzody merged 1 commit into
mainfrom
fix/trusted-server
May 6, 2025
Merged

fix: do not fetch remote collabora url for own server#4714
elzody merged 1 commit into
mainfrom
fix/trusted-server

Conversation

@elzody
Copy link
Copy Markdown
Collaborator

@elzody elzody commented Apr 29, 2025

  • Target version: main

Summary

When your own server is listed as a trusted server in the federated sharing options, there is no need to fetch the remote Collabora URL, as it would just be the one set internally.

TODO

  • Tests

Checklist

  • Code is properly formatted
  • Sign-off message is added to all commits
  • Documentation (manuals or wiki) has been updated or is not required

@elzody elzody added the 2. developing Work in progress label Apr 29, 2025
@elzody elzody self-assigned this Apr 29, 2025
@github-project-automation github-project-automation Bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Productivity team Apr 29, 2025
@elzody elzody moved this from 🧭 Planning evaluation (don't pick) to 🏗️ In progress in 📝 Productivity team Apr 29, 2025
Comment thread lib/Service/FederationService.php Outdated
Comment thread lib/Service/FederationService.php
@elzody elzody force-pushed the fix/trusted-server branch from 90616ae to 5d83307 Compare April 29, 2025 18:01
@elzody

This comment was marked as resolved.

@elzody elzody force-pushed the fix/trusted-server branch from 5d83307 to f205c9b Compare May 1, 2025 18:02
@elzody

This comment was marked as resolved.

@elzody elzody force-pushed the fix/trusted-server branch 3 times, most recently from 825e99d to baed008 Compare May 5, 2025 18:14
Signed-off-by: Elizabeth Danzberger <lizzy7128@tutanota.de>
@elzody elzody force-pushed the fix/trusted-server branch from baed008 to 399f04f Compare May 5, 2025 21:47
@elzody elzody marked this pull request as ready for review May 5, 2025 21:50
@elzody elzody moved this from 🏗️ In progress to 👀 In review in 📝 Productivity team May 5, 2025
@elzody
Copy link
Copy Markdown
Collaborator Author

elzody commented May 6, 2025

/backport to stable31

@elzody
Copy link
Copy Markdown
Collaborator Author

elzody commented May 6, 2025

/backport to stable30

@elzody
Copy link
Copy Markdown
Collaborator Author

elzody commented May 6, 2025

/backport to stable29

@elzody elzody merged commit 9075f51 into main May 6, 2025
69 checks passed
@elzody elzody deleted the fix/trusted-server branch May 6, 2025 17:40
@github-project-automation github-project-automation Bot moved this from 👀 In review to ☑️ Done in 📝 Productivity team May 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2. developing Work in progress

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants