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

Fallback URL for streaming API seems malformed #530

Closed
seism0saurus opened this issue Mar 4, 2025 · 2 comments · Fixed by #534
Closed

Fallback URL for streaming API seems malformed #530

seism0saurus opened this issue Mar 4, 2025 · 2 comments · Fixed by #534
Assignees
Labels
bug Something isn't working

Comments

@seism0saurus
Copy link

Hi,

First thank you very much for providing this library. I'm working on a social wall project for Mastodon and bigbone is a huge help.

During a try to implement an end to end test with Mastodon in a container i stumbled across the fallback URL for the streaming API. The fallback was triggered since I tried the LIMITED_FEREDATION_MODE in the Mastodon .env file. After enabling that, the /api/v2/instance endpoint is only accessable after authentication and so the fallback in your MastodonClient is used:

streamingUrl = getStreamingApiUrl(fallbackUrl = scheme + instanceName)

While debugging it became clear, that the constructed URL is missing the seperator between the scheme and the instanceName. E.g. if my container with mastodon is reachable via https as mymastodoninstance it is httpsmymastodoninstance. But it should be https://mymastodoninstance.

I'm not adapted to Kotlin yet so I can't provide a unit test at the moment. But maybe in the future.
Do you need any other information to verify this?

Kind regards
seism0saurus

@PattaFeuFeu PattaFeuFeu self-assigned this Mar 5, 2025
@PattaFeuFeu
Copy link
Owner

Hi seism0saurus,

thank you for creating the issue and raising the topic! I’ll have a look at it later. 😊

@PattaFeuFeu PattaFeuFeu added the bug Something isn't working label Mar 5, 2025
@PattaFeuFeu
Copy link
Owner

@seism0saurus The issue has been fixed. Thank you again for bringing it up.
We’re currently restructuring a bit so I cannot say yet when a new (SNAPSHOT) release will go live which contains the fix. I’ll make sure to mention you so that you get a notification.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants