Paths

Table of Contentst

[web] Use getMessageForException in shared worker connectHandler
ClosedPublic
Actions

Authored by ashoat on Apr 4 2024, 2:48 PM.

Tags

None

Referenced Files

	Unknown Object (File)
	Sun, Nov 3, 1:41 AM

	Unknown Object (File)
	Tue, Oct 22, 2:46 PM

	Unknown Object (File)
	Tue, Oct 22, 2:46 PM

	Unknown Object (File)
	Tue, Oct 22, 2:46 PM

	Unknown Object (File)
	Oct 11 2024, 11:49 AM

	Unknown Object (File)
	Sep 24 2024, 12:44 AM

	Unknown Object (File)
	Sep 20 2024, 4:27 PM

	Unknown Object (File)
	Aug 29 2024, 4:05 PM

View All 39 Files

Subscribers

tomek

Details

Reviewers

michal
will

Commits

rCOMM37b88717c320: [web] Use getMessageForException in shared worker connectHandler

Summary

This is safer, since technically in JS anything can be thrown (doesn't have to be an Error)

Test Plan

Flow

Diff Detail

Repository

rCOMM Comm

Branch

ashoat/except

Lint

No Lint Coverage

Unit

No Test Coverage

Event Timeline

ashoat created this revision.Apr 4 2024, 2:48 PM

Herald added a subscriber: tomek. · View Herald TranscriptApr 4 2024, 2:48 PM

Harbormaster completed remote builds in B28005: Diff 38814.Apr 4 2024, 3:04 PM

ashoat requested review of this revision.Apr 4 2024, 3:04 PM

will accepted this revision.Apr 4 2024, 9:48 PM

This revision is now accepted and ready to land.Apr 4 2024, 9:48 PM

Closed by commit rCOMM37b88717c320: [web] Use getMessageForException in shared worker connectHandler (authored by ashoat). · Explain WhyApr 5 2024, 6:32 AM

This revision was automatically updated to reflect the committed changes.

ashoat added a commit: rCOMM37b88717c320: [web] Use getMessageForException in shared worker connectHandler.

Revision Contents
Changeset List

Path

Size

web/

shared-worker/

worker/

shared-worker.js

4 lines

Diff 38814

View Options

web/shared-worker/worker/shared-worker.js

[web] Use getMessageForException in shared worker connectHandlerClosedPublicActions