cenobites / flask-jsonrpc

Basic JSON-RPC implementation for your Flask-powered sites

Geek Repo:Geek Repo

Github PK Tool:Github PK Tool

Feature reqest: do not allow to invoke method as JSON-RPC "Notification"

Talkless opened this issue · comments

I have spend hours debugging why I get 204 NO CONTENT when view does return data... Just because I've forgot to add "id:0" into JSON-RPC request...

It seem there's no way to check if it's notification or not? I would actually throw error if heavyweight request is ignored...

Maybe adding parameter like @jsonrpc_client.method("object.trip_stop", notification=False) could throw JSON-RPC error?

@Talkless please review the PR #385 for your considerations 🙏