Opened 9 months ago

Closed 3 months ago

#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)

Background

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.

Notes

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 9 months ago by mjethani

  • Cc hfiguiere jsonesen kzar sebastian greiner sergz added

comment:2 Changed 9 months ago by mjethani

  • Description modified (diff)

comment:3 Changed 9 months ago by mjethani

  • Cc Ross added

comment:4 Changed 3 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.