Skip to content

Commit

Permalink
bug fix: call access token method when creating or importing a wallet.
Browse files Browse the repository at this point in the history
  • Loading branch information
simonkernel committed Sep 12, 2022
1 parent 0f15519 commit 6f5966c
Show file tree
Hide file tree
Showing 3 changed files with 9 additions and 8 deletions.
13 changes: 7 additions & 6 deletions packages/auth/src/services/auth.js
Original file line number Diff line number Diff line change
Expand Up @@ -102,8 +102,8 @@ const build = async ({ seed, authMemberId, rpcEndpoint }) => {
throw new Error('already registered')
}
//TODO: change member owner to API
const { id: member_id, data: { role } } = await members.create({ wallet: iss, role: NEW_ROLE })
const registered = await wallets.create({ member_id, nickname }, { id: iss, owner: member_id })
const { id: memberId, data: { role } } = await members.create({ wallet: iss, role: NEW_ROLE })
const registered = await wallets.create({ memberId, nickname }, { id: iss, owner: memberId })

const authPayload = jwtService.authPayload({ iss, nickname, role })
return jwtService.createJwt(wallet, jwtService.AUTH_JWT, authPayload)
Expand All @@ -117,11 +117,12 @@ const build = async ({ seed, authMemberId, rpcEndpoint }) => {
if (!exists) {
member = await members.create({ wallet: iss, role: NEW_ROLE, groupIds: DEFAULT_GROUP_IDS })
await members.updateMeta(member.id, { owner: member.id })
const { id: member_id, data: { role } } = member
await wallets.create({ member_id, nickname }, { id: iss, owner: member_id })
const { id: memberId, data: { role } } = member
await wallets.create({ memberId, nickname }, { id: iss, owner: memberId })
} else {
const { data: { member_id } } = await wallets.get(iss)
member = await members.get(member_id)
// TODO: migrate entities from member_id to memberId
const { data: { member_id, memberId } } = await wallets.get(iss)
member = await members.get(memberId || member_id)
// TODO: quick fix to migrate legacy member entities
await members.updateMeta(member.id, { owner: member.id })
}
Expand Down
2 changes: 1 addition & 1 deletion packages/wallet/src/components/Create.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -65,7 +65,7 @@ const Create = () => {
wallet = null

// TODO: retry?
await client.call({ method: 'authService.register', params: [jwt] })
await client.call({ method: 'authService.accessToken', params: [jwt, true] })
} catch (error) {
console.error(error)
setErrorMessage(error.message)
Expand Down
2 changes: 1 addition & 1 deletion packages/wallet/src/components/Import.jsx
Original file line number Diff line number Diff line change
Expand Up @@ -66,7 +66,7 @@ const Import = () => {
wallet = null

// TODO: retry?
await client.call({ method: 'authService.register', params: [jwt] })
await client.call({ method: 'authService.accessToken', params: [jwt, true] })
} catch (error) {
setErrorMessage(error.message)
console.error(error)
Expand Down

0 comments on commit 6f5966c

Please sign in to comment.