share / sharedb
96%
master: 97%

LAST BUILD BRANCH: dependabot/bundler/docs/rexml-3.4.2
DEFAULT BRANCH: master
Repo Added
Files 39
Badge
Embed ▾
README BADGES
x

If you need to use a raster PNG badge, change the '.svg' to '.png' in the link

Markdown

Textile

RDoc

HTML

Rst

LAST BUILD ON BRANCH otapply-rollback
branch: otapply-rollback
CHANGE BRANCH
x

926

Pull #259

travis-ci

web-flow 
Update hard rollback callback logic

We previously followed the following logic for callbacks in a hard
rollback:

  - check there's an inflight op, and that it has a callback
  - if this isn't the case, then emit the error

This logic doesn't work very well in the situation where we don't have
an inflight op. This can happen in the following case:

  - I submit an invalid op
  - the op is added to the pending ops queue
  - we attempt to apply the op to the local document before flushing
  - `type.apply` throws, so the op is never flushed from pending to
    inflight
  - we perform a hard rollback
  - we now find we have no inflight op, but we do have a pending op

In this case, since we have no inflight op, we'd emit the error, but
also call the pending op callback, causing both a callback call _and_
an error emission, which is a bit surprising.

This change updates our logic, so that:

  - we check that there are some ops to callback
  - and that all of those ops have callbacks

If there are no callbacks, or if any of the ops don't handle the error,
then we emit, so that we can be sure of never swallowing an error. We
also avoid treating the inflight op any differently to pending ops, so
that we avoid behaviour that's dependent on where in its lifecycle an op
throws.

However, if the client handles all of their op submission errors, then
we never emit.
Pull Request #259: Catch synchronous errors in `Doc._otApply`

1093 of 1311 branches covered (83.37%)

18 of 18 new or added lines in 1 file covered. (100.0%)

2304 of 2410 relevant lines covered (95.6%)

407.65 hits per line

Relevant lines Covered
2410 RELEVANT LINES 2304 COVERED LINES
407.65 HITS PER LINE
Source Files on otapply-rollback
Coverage ∆ File Lines Relevant Covered Missed Hits/Line Branch Hits Branch Misses

Recent builds

Builds Branch Commit Type Ran Committer Via Coverage
926 otapply-rollback Update hard rollback callback logic We previously followed the following logic for callbacks in a hard rollback: - check there's an inflight op, and that it has a callback - if this isn't the case, then emit the error This logic doesn't wor... Pull #259 web-flow travis-ci pending completion  
925 otapply-rollback Update hard rollback callback logic We previously followed the following logic for callbacks in a hard rollback: - check there's an inflight op, and that it has a callback - if this isn't the case, then emit the error This logic doesn't wor... push Alec Gibson travis-ci pending completion  
922 otapply-rollback Catch synchronous errors in `Doc._otApply` Fixes https://github.com/share/sharedb/issues/257 As outlined in the above issue, calling `Doc.submitOp` can sometimes `throw` a synchronous error, for example when submitting an invalid op that causes ... Pull #259 web-flow travis-ci pending completion  
921 otapply-rollback Catch synchronous errors in `Doc._otApply` Fixes https://github.com/share/sharedb/issues/257 As outlined in the above issue, calling `Doc.submitOp` can sometimes `throw` a synchronous error, for example when submitting an invalid op that causes ... push Alec Gibson travis-ci pending completion  
920 otapply-rollback Catch synchronous errors in `Doc._otApply` Fixes https://github.com/share/sharedb/issues/257 As outlined in the above issue, calling `Doc.submitOp` can sometimes `throw` a synchronous error, for example when submitting an invalid op that causes ... Pull #259 web-flow travis-ci pending completion  
919 otapply-rollback Catch synchronous errors in `Doc._otApply` Fixes https://github.com/share/sharedb/issues/257 As outlined in the above issue, calling `Doc.submitOp` can sometimes `throw` a synchronous error, for example when submitting an invalid op that causes ... push Alec Gibson travis-ci pending completion  
918 otapply-rollback Catch synchronous errors in `Doc._otApply` Fixes https://github.com/share/sharedb/issues/257 As outlined in the above issue, calling `Doc.submitOp` can sometimes `throw` a synchronous error, for example when submitting an invalid op that causes ... Pull #259 web-flow travis-ci pending completion  
917 otapply-rollback Catch synchronous errors in `Doc._otApply` Fixes https://github.com/share/sharedb/issues/257 As outlined in the above issue, calling `Doc.submitOp` can sometimes `throw` a synchronous error, for example when submitting an invalid op that causes ... push Alec Gibson travis-ci pending completion  
See All Builds (1414)