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

Publish a BEP explaining how to add support for WebTorrent #881

Open
wants to merge 4 commits into
base: beps
from
Open
Changes from 1 commit
Commits
File filter...
Filter file types
Jump to…
Jump to file or symbol
Failed to load files and symbols.

Always

Just for now

Prev

Change the error response to have an action prop

  • Loading branch information
yciabaud committed Aug 29, 2016
commit 0544a16dd55845e785bc742861ae29ad9ea182c0
@@ -204,7 +204,8 @@ If the tracker encounters an error, it might send an error message.
error response::

{
"error": ""
"action": "error"
"message": ""
}

This comment has been minimized.

Copy link
@yciabaud

yciabaud Aug 29, 2016

Author Contributor

I changed the error response message for it to look like the other messages like in other beps.

This comment has been minimized.

Copy link
@wI2L

wI2L Oct 11, 2016

Tbh, I think that both failure reason and an extra error field are necesseray. The first one wpuld stay for legacy reasons and would be used only for Bittorrent protocol related failures (either announce or scrape) while another extra field (or object, with code/message ?) could describe a signalling error.



ProTip! Use n and p to navigate between commits in a pull request.
You can’t perform that action at this time.