refactor: Use an identity provider.

This commit is contained in:
Hayden Young 2023-09-07 09:38:05 +01:00
parent 06ea870308
commit c75b805ed1
5 changed files with 11 additions and 11 deletions

View File

@ -76,7 +76,7 @@ const Identities = async ({ keystore, path, storage, ipfs } = {}) => {
const identityProvider = await IdentityProvider() const identityProvider = await IdentityProvider()
if (!getIdentityProvider(identityProvider.type)) { if (!getIdentityProvider(identityProvider.type)) {
throw new Error('Identity provider is unknown. Use addIdentityProvider(provider) to register the identity provider') throw new Error('Identity provider is unknown. Use useIdentityProvider(provider) to register the identity provider')
} }
const id = await identityProvider.getId(options) const id = await identityProvider.getId(options)

View File

@ -7,7 +7,7 @@ export {
} from './identity.js' } from './identity.js'
export { export {
addIdentityProvider, useIdentityProvider,
getIdentityProvider, getIdentityProvider,
PublicKeyIdentityProvider PublicKeyIdentityProvider
} from './providers/index.js' } from './providers/index.js'

View File

@ -23,7 +23,7 @@ const getIdentityProvider = (type) => {
* @static * @static
* @memberof module:Identities * @memberof module:Identities
*/ */
const addIdentityProvider = (identityProvider) => { const useIdentityProvider = (identityProvider) => {
if (!identityProvider.type || if (!identityProvider.type ||
typeof identityProvider.type !== 'string') { typeof identityProvider.type !== 'string') {
throw new Error('Given IdentityProvider doesn\'t have a field \'type\'.') throw new Error('Given IdentityProvider doesn\'t have a field \'type\'.')
@ -40,6 +40,6 @@ const addIdentityProvider = (identityProvider) => {
identityProviders[identityProvider.type] = identityProvider identityProviders[identityProvider.type] = identityProvider
} }
addIdentityProvider(PublicKeyIdentityProvider) useIdentityProvider(PublicKeyIdentityProvider)
export { addIdentityProvider, getIdentityProvider, PublicKeyIdentityProvider } export { useIdentityProvider, getIdentityProvider, PublicKeyIdentityProvider }

View File

@ -31,7 +31,7 @@ export {
export { export {
Identities, Identities,
isIdentity, isIdentity,
addIdentityProvider, useIdentityProvider,
PublicKeyIdentityProvider PublicKeyIdentityProvider
} from './identities/index.js' } from './identities/index.js'

View File

@ -3,7 +3,7 @@ import rmrf from 'rimraf'
import { copy } from 'fs-extra' import { copy } from 'fs-extra'
import { toString as uint8ArrayToString } from 'uint8arrays/to-string' import { toString as uint8ArrayToString } from 'uint8arrays/to-string'
import KeyStore, { signMessage, verifyMessage } from '../../src/key-store.js' import KeyStore, { signMessage, verifyMessage } from '../../src/key-store.js'
import { Identities, addIdentityProvider, getIdentityProvider, Identity, PublicKeyIdentityProvider } from '../../src/identities/index.js' import { Identities, useIdentityProvider, getIdentityProvider, Identity, PublicKeyIdentityProvider } from '../../src/identities/index.js'
import testKeysPath from '../fixtures/test-keys-path.js' import testKeysPath from '../fixtures/test-keys-path.js'
import CustomIdentityProvider from '../fixtures/providers/custom.js' import CustomIdentityProvider from '../fixtures/providers/custom.js'
import FakeIdentityProvider from '../fixtures/providers/fake.js' import FakeIdentityProvider from '../fixtures/providers/fake.js'
@ -232,7 +232,7 @@ describe('Identities', function () {
}) })
it('false signature doesn\'t verify', async () => { it('false signature doesn\'t verify', async () => {
addIdentityProvider(FakeIdentityProvider) useIdentityProvider(FakeIdentityProvider)
identity = await identities.createIdentity({ provider: FakeIdentityProvider() }) identity = await identities.createIdentity({ provider: FakeIdentityProvider() })
const verified = await identities.verifyIdentity(identity) const verified = await identities.verifyIdentity(identity)
assert.strictEqual(verified, false) assert.strictEqual(verified, false)
@ -345,7 +345,7 @@ describe('Identities', function () {
describe('manage identity providers', () => { describe('manage identity providers', () => {
it('can add an identity provider', () => { it('can add an identity provider', () => {
addIdentityProvider(CustomIdentityProvider) useIdentityProvider(CustomIdentityProvider)
assert.deepStrictEqual(getIdentityProvider('custom'), CustomIdentityProvider) assert.deepStrictEqual(getIdentityProvider('custom'), CustomIdentityProvider)
}) })
@ -354,7 +354,7 @@ describe('Identities', function () {
let err let err
try { try {
addIdentityProvider(NoTypeIdentityProvider) useIdentityProvider(NoTypeIdentityProvider)
} catch (e) { } catch (e) {
err = e.toString() err = e.toString()
} }
@ -366,7 +366,7 @@ describe('Identities', function () {
let err let err
try { try {
addIdentityProvider(NoVerifyIdentityIdentityProvider) useIdentityProvider(NoVerifyIdentityIdentityProvider)
} catch (e) { } catch (e) {
err = e.toString() err = e.toString()
} }