Opened 2 months ago

Last modified 2 months ago

#7393 new change

Do error handling for fallback URL

Reported by: mjethani Assignee:
Priority: Unknown Milestone:
Module: Core Keywords:
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 (3)

comment:1 Changed 2 months ago by mjethani

  • Cc hfiguiere jsonesen kzar sebastian greiner sergz added

comment:2 Changed 2 months ago by mjethani

  • Description modified (diff)

comment:3 Changed 2 months ago by mjethani

  • Cc Ross added
Note: See TracTickets for help on using tickets.