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

feat: add runOnTransientConnection option to pass to registrar #485

Merged
merged 1 commit into from
Jan 29, 2024

Conversation

achingbrain
Copy link
Collaborator

Adds an option to allow running gossipsub over transient (e.g. relayed) connections.

By default this is not allowed since transient connections are by design data/time limited and breaching those limits can cause the connection to be closed abruptly, but sometimes people control the relay servers and disable the limits so allow them to use gossipsub over relayed connections if they choose.

@achingbrain achingbrain requested a review from a team as a code owner January 26, 2024 14:16
@achingbrain achingbrain force-pushed the fix/add-run-on-transient-option branch from 6fc011f to d5d2329 Compare January 26, 2024 14:21
Adds an option to allow running gossipsub over transient (e.g. relayed)
connections.

By default this is not allowed since transient connections are by
design data/time limited and breaching those limits can cause the
connection to be closed abruptly, but sometimes people control the
relay servers and disable the limits so allow them to use gossipsub
over relayed connections if they choose.
@wemeetagain wemeetagain merged commit 986ff6c into master Jan 29, 2024
9 checks passed
@wemeetagain wemeetagain deleted the fix/add-run-on-transient-option branch January 29, 2024 19:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants