#7393 closed change (rejected)

Do error handling for fallback URL

Reported by: mjethani Assignee:
Priority: Unknown Milestone:
Module: Core Keywords: closed-in-favor-of-gitlab
Cc: hfiguiere, jsonesen, kzar, sebastian, greiner, sergz, Ross Blocked By:
Blocking: Platform: Unknown / Cross platform
Ready: no Confidential: no
Tester: Unknown Verified working: no
Review URL(s):

Description (last modified by mjethani)


Currently there is no error handling for the fallback URL (see code in lib/synchronizer.js). It always assumes that the request succeeds. If the request fails, it should be handled. Additionally, there should be some tests for this.

What to change

To be determined.


We appear not to have any tests for the fallback URL feature (see ticket:7376#comment:15). We should add tests.

Change History (4)

comment:1 Changed 17 months ago by mjethani

  • Cc hfiguiere jsonesen kzar sebastian greiner sergz added

comment:2 Changed 17 months ago by mjethani

  • Description modified (diff)

comment:3 Changed 17 months ago by mjethani

  • Cc Ross added

comment:4 Changed 12 months ago by sebastian

  • Keywords closed-in-favor-of-gitlab added
  • Resolution set to rejected
  • Status changed from new to closed

Sorry, but we switched to GitLab. If this issue is still relevant, please file it again in the new issue tracker.

Note: See TracTickets for help on using tickets.