[lib] add Tunnelbroker request timeout
ClosedPublic
Actions

Authored by kamil on Fri, Sep 6, 4:26 AM.

Tags

None

Referenced Files

	F2708504: D13254.diff
	Sun, Sep 15, 3:11 PM

	F2708482: D13254.id44006.diff
	Sun, Sep 15, 3:00 PM

	F2708064: D13254.id43912.diff
	Sun, Sep 15, 1:28 PM

	F2707713: D13254.id43912.diff
	Sun, Sep 15, 12:54 PM

	Unknown Object (File)
	Sun, Sep 15, 6:37 AM

	Unknown Object (File)
	Sat, Sep 14, 8:32 PM

	Unknown Object (File)
	Sat, Sep 14, 7:58 AM

	Unknown Object (File)
	Thu, Sep 12, 3:42 AM

View All 11 Files

Subscribers

ashoat

Details

Reviewers

tomek
bartek

Commits

rCOMM4154d1237b68: [lib] add Tunnelbroker request timeout

Summary

ENG-9105.

Adding timeout to reject promise, this is similar timeout approach to other places in our codebase.

Test Plan

Simulate hanging Tunnelbroker request and check that promise is rejected on the client.

Diff Detail

Repository

rCOMM Comm

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Fri, Sep 6, 4:26 AM

kamil held this revision as a draft.

Herald added a subscriber: ashoat. · View Herald TranscriptFri, Sep 6, 4:26 AM

Harbormaster completed remote builds in B31447: Diff 43912.Fri, Sep 6, 4:47 AM

kamil published this revision for review.Fri, Sep 6, 5:37 AM

tomek accepted this revision.Fri, Sep 6, 7:48 AM

This revision is now accepted and ready to land.Fri, Sep 6, 7:48 AM

Closed by commit rCOMM4154d1237b68: [lib] add Tunnelbroker request timeout (authored by kamil). · Explain WhyTue, Sep 10, 5:51 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM4154d1237b68: [lib] add Tunnelbroker request timeout.

Revision Contents
Changeset List

Path

Size

lib/

shared/

timeouts.js

3 lines

tunnelbroker/

tunnelbroker-context.js

15 lines

Diff 44006

View Options

lib/shared/timeouts.js