Skip to content
This repository has been archived by the owner on Nov 15, 2023. It is now read-only.

Rename system_networkState to system_unstable_networkState #8001

Merged
1 commit merged into from
Feb 2, 2021

Conversation

tomaka
Copy link
Contributor

@tomaka tomaka commented Jan 28, 2021

Renames this JSON-RPC function in order to make it clear that has never been intended to be stable nor long-term.

This is a breaking change. If you had an automatic script that is using system_networkState, which you weren't supposed to do in the first place, then it will break.

If you were using this function to get the local PeerId and listened addresses, then I invite you to use system_localPeerId and system_localListenAddresses instead.

cc paritytech/polkadot-sdk#558

@tomaka tomaka added A0-please_review Pull request needs code review. B5-clientnoteworthy C1-low PR touches the given topic and has a low impact on builders. labels Jan 28, 2021
@dvdplm
Copy link
Contributor

dvdplm commented Jan 28, 2021

@maciejhirsz Does Telemetry use this one?

@tomaka
Copy link
Contributor Author

tomaka commented Jan 29, 2021

Nodes send this "network state" to the telemetry, which then shows it in the UI.
The telemetry however doesn't do any JSON-RPC call (unless you're talking about a different telemetry), so it's not affected.

@tomaka
Copy link
Contributor Author

tomaka commented Feb 2, 2021

Could someone else approve this PR?

@romanb romanb self-requested a review February 2, 2021 14:35
@tomaka
Copy link
Contributor Author

tomaka commented Feb 2, 2021

Thanks!

@tomaka
Copy link
Contributor Author

tomaka commented Feb 2, 2021

bot merge

@ghost
Copy link

ghost commented Feb 2, 2021

Trying merge.

@ghost ghost merged commit bc69520 into paritytech:master Feb 2, 2021
@tomaka tomaka deleted the mv-network-state branch February 2, 2021 15:04
This pull request was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A0-please_review Pull request needs code review. C1-low PR touches the given topic and has a low impact on builders.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants