Skip to content
This repository has been archived by the owner on Sep 4, 2024. It is now read-only.

Bearer Tokens and support for top-level named params #23

Open
ddimaria opened this issue Mar 15, 2019 · 1 comment
Open

Bearer Tokens and support for top-level named params #23

ddimaria opened this issue Mar 15, 2019 · 1 comment

Comments

@ddimaria
Copy link

ddimaria commented Mar 15, 2019

Hello, I just wanted to put a note about a fork I did yesterday (https://github.com/ddimaria/rust-jsonrpc). I altered the authentication to support Bearer tokens and removed the dependency on Vec params b/c I had to target a jsonrpc server that didn't conform to batch (Vec) requests and had a diff authentication protocol. I plan to abstract the auth methods and generalize the params parameters to allow for both use cases and submit a PR back to the main repo.

@tcharding
Copy link
Collaborator

Hi @ddimaria, can this issue be closed now? I'm a bit late to the party but thanks for letting us know.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants