Skip to content

Commit e826e2f

Browse files
authored
fix([sc-15475]): extended popup transport establishTimeout to 1 min (#251)
fix([sc-16068]): standards and demo page titles
1 parent 99166ef commit e826e2f

File tree

4 files changed

+25
-24
lines changed

4 files changed

+25
-24
lines changed

examples/react-dapp/index.html

Lines changed: 16 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -1,22 +1,18 @@
11
<!doctype html>
22
<html lang="en">
3-
<head>
4-
<meta charset="UTF-8" />
5-
<link rel="icon" type="image/svg+xml" href="/favicon.ico" />
6-
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
7-
<!-- <meta name="color-scheme" content="dark"> -->
8-
<title>Identity Kit Playground</title>
9-
<link
10-
href="https://fonts.googleapis.com/css2?family=IBM+Plex+Mono&display=swap"
11-
rel="stylesheet"
12-
/>
13-
<link
14-
href="https://fonts.googleapis.com/css2?family=Inter:wght@400;600;700&display=swap"
15-
rel="stylesheet"
16-
/>
17-
</head>
18-
<body>
19-
<div id="root"></div>
20-
<script type="module" src="/src/main.tsx"></script>
21-
</body>
22-
</html>
3+
4+
<head>
5+
<meta charset="UTF-8" />
6+
<link rel="icon" type="image/svg+xml" href="/favicon.ico" />
7+
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
8+
<title>NFID IdentityKit | Standards</title>
9+
<link href="https://fonts.googleapis.com/css2?family=IBM+Plex+Mono&display=swap" rel="stylesheet" />
10+
<link href="https://fonts.googleapis.com/css2?family=Inter:wght@400;600;700&display=swap" rel="stylesheet" />
11+
</head>
12+
13+
<body>
14+
<div id="root"></div>
15+
<script type="module" src="/src/main.tsx"></script>
16+
</body>
17+
18+
</html>

examples/react-demo/index.html

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,7 @@
44
<meta charset="UTF-8" />
55
<link rel="icon" type="image/svg+xml" href="/favicon.ico" />
66
<meta name="viewport" content="width=device-width, initial-scale=1.0" />
7-
<title>Identity Kit Playground</title>
7+
<title>NFID IdentityKit | Demo</title>
88
<link
99
href="https://fonts.googleapis.com/css2?family=IBM+Plex+Mono&display=swap"
1010
rel="stylesheet"

packages/identitykit/src/lib/service/transport-builder/new-tab-transport.builder.spec.ts

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,8 @@
11
import { PostMessageTransport, PostMessageTransportOptions } from "@slide-computer/signer-web"
2-
import { getPopupTransportBuilder } from "./new-tab-transport.builder"
2+
import {
3+
getPopupTransportBuilder,
4+
NEW_TAB_TRANSPORT_DEFAULT_ESTABLISH_TIMEOUT,
5+
} from "./new-tab-transport.builder"
36

47
jest.mock("@slide-computer/signer-web", () => ({
58
PostMessageTransport: jest.fn(),
@@ -25,7 +28,7 @@ describe("getPopupTransportBuilder", () => {
2528
await getPopupTransportBuilder(request)
2629
expect(PostMessageTransport).toHaveBeenCalledWith({
2730
url: mockUrl,
28-
establishTimeout: 15000,
31+
establishTimeout: NEW_TAB_TRANSPORT_DEFAULT_ESTABLISH_TIMEOUT,
2932
})
3033
})
3134
})
Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,13 @@
11
import { Transport } from "@slide-computer/signer"
22
import { PostMessageTransport, PostMessageTransportOptions } from "@slide-computer/signer-web"
33

4+
export const NEW_TAB_TRANSPORT_DEFAULT_ESTABLISH_TIMEOUT = 60000 // 1 min
5+
46
export async function getPopupTransportBuilder(
57
options: PostMessageTransportOptions
68
): Promise<Transport> {
79
return new PostMessageTransport({
810
...options,
9-
establishTimeout: 15000,
11+
establishTimeout: NEW_TAB_TRANSPORT_DEFAULT_ESTABLISH_TIMEOUT, // 1 min
1012
})
1113
}

0 commit comments

Comments
 (0)