Skip to content

Conversation

@martinconic
Copy link
Contributor

… (#5069)"

This reverts commit 39e9dc8.

Checklist

  • I have read the coding guide.
  • My change requires a documentation update, and I have done it.
  • I have added tests to cover my changes.
  • I have filled out the description and linked the related issues.

Description

Open API Spec Version Changes (if applicable)

Motivation and Context (Optional)

Related Issue (Optional)

Screenshots (if appropriate):

@acha-bill acha-bill changed the title Revert "fix: update libp2p and use share listen address functionality… revert: "fix: update libp2p and use share listen address functionality… May 22, 2025
@martinconic martinconic merged commit e188a40 into master May 22, 2025
15 checks passed
@martinconic martinconic deleted the revert-commit-39e9dc81 branch May 22, 2025 12:15
@janos
Copy link
Member

janos commented May 22, 2025

I believe that this PR #5095 also needs to be reverted as websockets are not functional without shared tcp, and there is no need for config variable to be set to enabled, buy default.

@acha-bill
Copy link
Contributor

acha-bill commented May 22, 2025

I don't think there's any impact on enabling WS by default.
We are going to figure out the shared TCP thing in the next release anyways.

Lets revert it

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.

5 participants