Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Handle redirected servers in server detection. #14819

Merged
merged 2 commits into from
Feb 16, 2025

Conversation

annando
Copy link
Collaborator

@annando annando commented Feb 15, 2025

This handles a problem with services like brid.gy, where urls like bsky.brid.gy are a redirect to fed.brid.gy. This formerly lead to the problem that posts from that host hadn't been displayed with the correct icon.

See snarfed/bridgy-fed#1769

@MrPetovan MrPetovan merged commit c746017 into friendica:develop Feb 16, 2025
29 checks passed
@annando annando deleted the redirected branch February 16, 2025 05:24
annando added a commit to annando/friendica that referenced this pull request Feb 16, 2025
@AlfredSK
Copy link

I get an error when running postupdate manually.

sudo -u www-data php bin/console.php postupdate
Check for pending update actions.
ALTER IGNORE TABLE `gserver` DROP INDEX `redirect-gsid`; 

Error 1553 occurred during database update:
Cannot drop index 'redirect-gsid': needed in a foreign key constraint
Done.

annando added a commit to annando/friendica that referenced this pull request Feb 16, 2025
annando added a commit to annando/friendica that referenced this pull request Feb 16, 2025
@annando
Copy link
Collaborator Author

annando commented Feb 16, 2025

@AlfredSK see #14823

tobiasd added a commit that referenced this pull request Feb 16, 2025
@MrPetovan MrPetovan added this to the 2025.02 milestone Feb 17, 2025
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.

3 participants