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

Whisper rpc types #189

Merged
merged 3 commits into from
Dec 5, 2018
Merged

Whisper rpc types #189

merged 3 commits into from
Dec 5, 2018

Conversation

coffeepots
Copy link
Contributor

Small PR that:

  • Updates the whisper types and FilterOptions to be optional
  • Removs the duplicated WhisperIdentity from rpc_types.nim as it's already present in eth_common.eth_types.
  • Adds WhisperFilterOptions
  • Updates the signature for shh_newFilter to use WhisperFilterOptions rather than the eth FilterOptions

@zah
Copy link
Contributor

zah commented Dec 5, 2018

@kdeme, Ryan (@coffeepots) wasn't aware that you'll be also working on this. He'll be happy to leave the rest of the integration work to you.

@coffeepots
Copy link
Contributor Author

Yes, these changes are just updates to data structures really. In particular shh_newFilter was incorrect so that had to change anyway.

@kdeme
Copy link
Contributor

kdeme commented Dec 5, 2018

OK, that's fine.

I was however planning of changing the RPC calls to the list here: https://github.com/ethereum/go-ethereum/wiki/Whisper-v6-RPC-API

I assumed that this is the latest version and that https://github.com/ethereum/wiki/wiki/JSON-RPC is not up to date with regards to Whisper v6.

Or where should I be looking for the latest correct Whisper RPC API?

There is this EIP in the making, but Whisper is not included there.

@zah
Copy link
Contributor

zah commented Dec 5, 2018

The v6 RPCs would indeed be the most useful ones. It's what the Status app relies on.

@zah zah merged commit 51df75a into master Dec 5, 2018
@coffeepots
Copy link
Contributor Author

Yes I was working with https://github.com/ethereum/wiki/wiki/JSON-RPC so it looks like all of the RPC stubs for whisper are out of date now.

@coffeepots coffeepots deleted the whisper-rpc-types branch December 5, 2018 17:33
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