jonny
/
Signal-Desktop
1
0
Fork 0
Code Issues Pull Requests Packages Projects Releases Wiki Activity

Update to new provisioning URL scheme

This commit is contained in:
Evan Hahn 2021-09-09 17:08:19 -05:00 committed by GitHub
parent a67335272c
commit 8064560a5f
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23
2 changed files with 10 additions and 7 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

8
ts/test-both/util/getProvisioningUrl_test.ts → ts/test-node/util/getProvisioningUrl_test.ts
View File

@ -8,6 +8,10 @@ import { typedArrayToArrayBuffer } from '../../Crypto';
import { getProvisioningUrl } from '../../util/getProvisioningUrl';
// It'd be nice to run these tests in the renderer, too, but [Chromium's `URL` doesn't
// handle `sgnl:` links correctly][0].
//
// [0]: https://bugs.chromium.org/p/chromium/issues/detail?id=869291
describe('getProvisioningUrl', () => {
it('returns a URL with a UUID and public key', () => {
const uuid = 'a08bf1fd-1799-427f-a551-70af747e3956';
@ -16,8 +20,8 @@ describe('getProvisioningUrl', () => {
const result = getProvisioningUrl(uuid, typedArrayToArrayBuffer(publicKey));
const resultUrl = new URL(result);
assert(result.startsWith('tsdevice:/?'));
assert.strictEqual(resultUrl.protocol, 'tsdevice:');
assert.strictEqual(resultUrl.protocol, 'sgnl:');
assert.strictEqual(resultUrl.host, 'linkdevice');
assert.strictEqual(size(resultUrl.searchParams.entries()), 2);
assert.strictEqual(resultUrl.searchParams.get('uuid'), uuid);
assert.strictEqual(resultUrl.searchParams.get('pub_key'), 'CQgHBgUEAw==');

9
ts/util/getProvisioningUrl.ts
View File

@ -7,9 +7,8 @@ export function getProvisioningUrl(
uuid: string,
publicKey: ArrayBuffer
): string {
const params = new URLSearchParams({
uuid,
pub_key: arrayBufferToBase64(publicKey),
});
return `tsdevice:/?${params.toString()}`;
const url = new URL('sgnl://linkdevice');
url.searchParams.set('uuid', uuid);
url.searchParams.set('pub_key', arrayBufferToBase64(publicKey));
return url.toString();
}