Paths

Table of Contentst

Differential D12912

[lib] handle `BadDeviceToken` message
ClosedPublic
Actions

Authored by kamil on Jul 29 2024, 2:56 AM.

Tags

None

Referenced Files

	Unknown Object (File)
	Thu, Dec 19, 10:39 AM

	Unknown Object (File)
	Thu, Dec 19, 10:39 AM

	Unknown Object (File)
	Thu, Dec 19, 10:39 AM

	Unknown Object (File)
	Thu, Dec 19, 10:39 AM

	Unknown Object (File)
	Tue, Nov 26, 3:25 AM

	Unknown Object (File)
	Nov 22 2024, 1:24 PM

	Unknown Object (File)
	Nov 9 2024, 11:36 PM

	Unknown Object (File)
	Nov 9 2024, 4:15 PM

View All 63 Files

Subscribers

Details

Reviewers

bartek
marcin
tomek

Commits

rCOMM10b64a112df7: [lib] handle `BadDeviceToken` message

Summary

Updating state after receiving message from TB.

Depends on D12911

Test Plan

Repeat test plan from D12911

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

kamil created this revision.Jul 29 2024, 2:56 AM

kamil held this revision as a draft.

Herald added subscribers: tomek, ashoat. · View Herald TranscriptJul 29 2024, 2:56 AM

Harbormaster completed remote builds in B30736: Diff 42864.Jul 29 2024, 3:18 AM

kamil added a child revision: D12915: [web][desktop] refresh device token after receiving message from Tunnelbroker.Jul 29 2024, 4:07 AM

kamil added a child revision: D12916: [native] refresh device token after receiving message from Tunnelbroker.

kamil edited the summary of this revision. (Show Details)Jul 29 2024, 6:38 AM

kamil edited the test plan for this revision. (Show Details)

kamil published this revision for review.Jul 29 2024, 6:42 AM

kamil added a reviewer: tomek.

kamil added inline comments.

lib/reducers/tunnelbroker-device-token-reducer.js
37–38 ↗	(On Diff #42864)	We set `tunnelbrokerToken` to `null` because when receiving `BAD_DEVICE_TOKEN` Tunnelbroker is no longer trying to use that device token - so we can assume Tunnelbroker doesn't have any token. Additionally, this makes `TunnelbrokerDeviceTokenHandler` to upload token even if the same token was retrieved by native API (I think there is a really low chance for this, but still worth handling).

kamil mentioned this in D12910: [Tunnelbroker][lib] add `BadDeviceToken` message type.Jul 29 2024, 8:26 AM

tomek accepted this revision.Jul 30 2024, 2:22 AM

This revision is now accepted and ready to land.Jul 30 2024, 2:22 AM

Closed by commit rCOMM10b64a112df7: [lib] handle `BadDeviceToken` message (authored by kamil). · Explain WhyAug 1 2024, 1:28 AM

This revision was automatically updated to reflect the committed changes.

kamil added a commit: rCOMM10b64a112df7: [lib] handle `BadDeviceToken` message.

ashoat added inline comments.Aug 1 2024, 1:31 PM

lib/actions/tunnelbroker-actions.js
9	We have a convention that this name should end in `ActionType`

kamil added a child revision: D12968: [lib] rename `invalidateTunnelbrokerDeviceToken` -> `invalidateTunnelbrokerDeviceTokenActionType`.Aug 5 2024, 3:53 AM

kamil added inline comments.Aug 5 2024, 3:57 AM

lib/actions/tunnelbroker-actions.js
9	thanks for catching: D12968

Revision Contents
Changeset List

Path

Size

lib/

actions/

tunnelbroker-actions.js

3 lines

reducers/

tunnelbroker-device-token-reducer.js

15 lines

tunnelbroker/

use-peer-to-peer-message-handler.js

13 lines

types/

8 lines

Diff 43010

lib/actions/tunnelbroker-actions.js

Loading...

lib/reducers/tunnelbroker-device-token-reducer.js

Loading...

lib/tunnelbroker/use-peer-to-peer-message-handler.js

Loading...

lib/types/redux-types.js

Loading...