Skip to content

Commit 2411c71

Browse files
authored
fix!: require eventSource in PostMessageTransport + add security tests (#208)
1 parent c7ef5f0 commit 2411c71

File tree

3 files changed

+433
-2
lines changed

3 files changed

+433
-2
lines changed

src/message-transport.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -73,7 +73,7 @@ export class PostMessageTransport implements Transport {
7373
*/
7474
constructor(
7575
private eventTarget: Window = window.parent,
76-
private eventSource?: MessageEventSource,
76+
private eventSource: MessageEventSource,
7777
) {
7878
this.messageListener = (event) => {
7979
if (eventSource && event.source !== this.eventSource) {

src/react/useApp.tsx

Lines changed: 4 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -117,7 +117,10 @@ export function useApp({
117117

118118
async function connect() {
119119
try {
120-
const transport = new PostMessageTransport(window.parent);
120+
const transport = new PostMessageTransport(
121+
window.parent,
122+
window.parent,
123+
);
121124
const app = new App(appInfo, capabilities);
122125

123126
// Register handlers BEFORE connecting

0 commit comments

Comments
 (0)