Skip to content

feat: enable IP version specification for realtime app#253

Merged
w3b6x9 merged 1 commit intosupabase:masterfrom
dudlf:feat/realtime-ipv
Sep 26, 2022
Merged

feat: enable IP version specification for realtime app#253
w3b6x9 merged 1 commit intosupabase:masterfrom
dudlf:feat/realtime-ipv

Conversation

@dudlf
Copy link
Copy Markdown

@dudlf dudlf commented Apr 29, 2022

What kind of change does this PR introduce?

feature

What is the current behavior?

Machine which does not support IPv6 can't run realtime due to there is no options for this.

What is the new behavior?

User can specify IP version for realtime app.
Default behavior does not change.

Additional context

I'm new to supabase, and does not have any experience on Elixir and Phoenix.
Is there any possible issue when realtime is binded to IPv4?

@w3b6x9
Copy link
Copy Markdown
Member

w3b6x9 commented May 3, 2022

@dudlf are you currently having issues connecting Realtime server via IPv4? are you self-hosting?

@dudlf
Copy link
Copy Markdown
Author

dudlf commented May 9, 2022

@dudlf are you currently having issues connecting Realtime server via IPv4? are you self-hosting?

I'm using self-hosting.

@Silur
Copy link
Copy Markdown

Silur commented Aug 16, 2022

self hosting doesn't work on systems where ipv6 is explicitly disabled (by sysctl or kernel flags), so an ip family flag/setting is indeed preferred

@TeeWallz
Copy link
Copy Markdown

TeeWallz commented Sep 5, 2022

I experienced the issue this resolves today,

This PR would be really appreciated

Copy link
Copy Markdown
Member

@w3b6x9 w3b6x9 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@dudlf thanks for your contribution!

@w3b6x9 w3b6x9 merged commit 759860a into supabase:master Sep 26, 2022
@github-actions
Copy link
Copy Markdown

🎉 This PR is included in version 0.25.0 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

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

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants