Skip to content

Never ever abort sync runs / check that the connection is indeed broken before aborting on a timeout #5859

Closed

Description

@michaelstingl

Some server side errors cause the client to abort the current sync run. If the ownCloud admin or user is not able to fix the error, EVERY sync run will abort at the same position, and following files that don't have an error will NEVER sync because of this.

@SamuAlfageme We discussed this a time ago. Please link existing issues here or open new ones that could could cause this behaviour. Please also elaborate on the server improvements.

Activity

  1. guruz commented on Jun 26, 2017

    @guruz
    Contributor

    Please report only if this happens with latest client version as this is something which changed a lot back and forth :-)
    (CC @ckamm @ogoffart )

  2. added this to the 2.4.0 milestone on Jun 26, 2017
  3. SamuAlfageme commented on Jun 26, 2017

    @SamuAlfageme
    Contributor

    WIP

    Sync aborted - on server/network error

    Sync aborted - on client/protocol error

    Related Fixes

  4. ckamm commented on Jun 28, 2017

    @ckamm
    Contributor

    Incomplete list of potential sync run aborts during propagation stage, by looking at FatalError in the code

    • can't write to database
    • network error > NoError and <= UnknownProxyError
    • network error 503
    • timeout on download
    • critical disk space error on download
  5. ogoffart commented on Jul 21, 2017

    @ogoffart
    Contributor

    I believe the current behavior is the correct one: We only abort the sync run if we experience fatal error (network down, server on maintainnence, ...)

    @michaelstingl Do you have any concrete example of error that causes the sync to abort while it should not?

  6. removed this from the 2.4.0 milestone on Jul 28, 2017
  7. 23 remaining items

  8. SamuAlfageme commented on Jun 1, 2018

    @SamuAlfageme
    Contributor
  9. michaelstingl commented on Oct 18, 2018

    @michaelstingl
    ContributorAuthor

    Another candidate: #6826

  10. HanaGemela commented on Jun 3, 2019

    @HanaGemela
    Contributor

    @ogoffart @michaelstingl what exactly is ready to test here? Only abortion on error 503?

  11. ogoffart commented on Jun 4, 2019

    @ogoffart
    Contributor

    Right, when the server returns an error 503 for a file, (but the server is not in maintainence mode) it should still sync all other files.
    I don't know exactly how to get error 503 on the server, this is usually because of a problem in the storage backend.
    However, when the server is in maintainence mode, the error is also 503, but in this case, the sync needs to stop. (because otherwise all the files would be 503)

    Maybe we can just close this issue.

  12. HanaGemela commented on Jul 30, 2019

    @HanaGemela
    Contributor

    Closing this aggregating issue. All individual issues will be/were tested separately. 503 error will be tested in #5088

  13. michaelstingl commented on Jul 30, 2019

    @michaelstingl
    ContributorAuthor

    Closing this aggregating issue. All individual issues will be/were tested separately.

    👍 😍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

Type

No type

Projects

No projects

    Milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions