Skip to content

/ webtorrent

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

Defer Store Verification #1350

Open
wants to merge 5 commits into
base: master
from
Open

Defer Store Verification #1350

wants to merge 5 commits into from
+69 −46
Commits on May 8, 2018

  1. Defer store verification

    @KayleePop
    KayleePop committed Apr 11, 2018 
    Allow file access before the entire store has been validated.

Critical pieces are validated immediately instead of waiting in the
parallelLimit() queue.
    11ced94

  2. rename _markVerified() to _markInStore()

    @KayleePop
    KayleePop committed Apr 11, 2018 
    This avoids confusion with setting an index in _verified[] to true.
    9249d80

  3. Remove _onStore() function

    @KayleePop
    KayleePop committed Apr 11, 2018 
    This function isn't useful anymore because store verification happens
after the 'ready' event.
    35aeb52

  4. Clarify debug and error messages regarding piece verification

    @KayleePop
    KayleePop committed Apr 11, 2018
    Loading status checks…
     
    Specify whether the verification was on a piece in the store, or on a piece from a wire.
    40a67a5
Commits on Jun 18, 2018

  1. Don't download critical pieces before verification

    @KayleePop
    KayleePop committed Jun 18, 2018
    Loading status checks…
     
    This makes metadata always available before every request, and race
conditions between verification and downloading aren't possible.

Critical pieces are still prioritized for verification.
    5d1934b
You can’t perform that action at this time.