fix: fix backward compatibility for did registry interface

energywebfoundation · May 17, 2022 · a24a91c · a24a91c
1 parent 434e262
commit a24a91c
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 24 deletions.
diff --git a/src/modules/did-registry/did-registry.service.ts b/src/modules/did-registry/did-registry.service.ts
@@ -134,12 +134,11 @@ export class DidRegistry {
    * @param {GetDIDDocumentOptions} options object with options
    * @return DID document
    */
-  async getDidDocument(
-    { did, includeClaims }: GetDIDDocumentOptions = {
-      did: this._did,
-      includeClaims: true,
-    }
-  ): Promise<IDIDDocument> {
+  async getDidDocument({
+    did = this._did,
+    includeClaims = true,
+  }: GetDIDDocumentOptions = {}): Promise<IDIDDocument> {
+    getLogger().info(`Getting DID document for ${did}`);
     if (this._cacheClient) {
       try {
         const didDoc = await this._cacheClient.getDidDocument(
@@ -179,11 +178,9 @@ export class DidRegistry {
    * @param {GetServicesOptions} options object with options
    * @returns list of claims
    */
-  async getServices(
-    { did }: GetServicesOptions = {
-      did: this._signerService.did,
-    }
-  ): Promise<IServiceEndpoint[]> {
+  async getServices({
+    did = this._signerService.did,
+  }: GetServicesOptions = {}): Promise<IServiceEndpoint[]> {
     const didDocument = await this.getDidDocument({ did });
     return didDocument?.service;
   }
@@ -199,11 +196,9 @@ export class DidRegistry {
    * @param {GetDidPublicKeysOptions} options object with options
    * @returns list of public keys
    */
-  async getDidPublicKeys(
-    { did }: GetDidPublicKeysOptions = {
-      did: this._signerService.did,
-    }
-  ): Promise<IPublicKey[]> {
+  async getDidPublicKeys({
+    did = this._signerService.did,
+  }: GetDidPublicKeysOptions = {}): Promise<IPublicKey[]> {
     const didDocument = await this.getDidDocument({ did });
     return didDocument?.publicKey;
   }
@@ -219,11 +214,9 @@ export class DidRegistry {
    * @param {GetDidDelegatesOptions} options object with options
    * @returns list of delegates
    */
-  async getDidDelegates(
-    { did }: GetDidDelegatesOptions = {
-      did: this._did,
-    }
-  ): Promise<string[] | undefined> {
+  async getDidDelegates({
+    did = this._signerService.did,
+  }: GetDidDelegatesOptions = {}): Promise<string[] | undefined> {
     const didDocument = await this.getDidDocument({ did });
     return didDocument?.delegates;
   }

diff --git a/src/modules/did-registry/did.types.ts b/src/modules/did-registry/did.types.ts
@@ -40,17 +40,17 @@ export interface GetDIDDocumentOptions {
 
 export interface GetServicesOptions {
   /* DID of the user */
-  did: string;
+  did?: string;
 }
 
 export interface GetDidPublicKeysOptions {
   /* DID of the user */
-  did: string;
+  did?: string;
 }
 
 export interface GetDidDelegatesOptions {
   /* DID of the user */
-  did: string;
+  did?: string;
 }
 
 export interface CreatePublicClaimOptions {