more url encodings for alias

provenant-dev · Apr 19, 2024 · 6fb6f34 · 6fb6f34
1 parent fcfca8c
commit 6fb6f34
Show file tree

Hide file tree

Showing 5 changed files with 15 additions and 15 deletions.
diff --git a/examples/integration-scripts/utils/test-setup.ts b/examples/integration-scripts/utils/test-setup.ts
@@ -121,8 +121,8 @@ export async function getEndRoles(
 ): Promise<any> {
     const path =
         role !== undefined
-            ? `/identifiers/${alias}/endroles/${role}`
-            : `/identifiers/${alias}/endroles`;
+            ? `/identifiers/${encodeURIComponent(alias)}/endroles/${role}`
+            : `/identifiers/${encodeURIComponent(alias)}/endroles`;
     const response: Response = await client.fetch(path, 'GET', null);
     if (!response.ok) throw new Error(await response.text());
     const result = await response.json();

diff --git a/src/keri/app/coring.ts b/src/keri/app/coring.ts
@@ -33,7 +33,7 @@ export class Oobis {
      * @returns {Promise<any>} A promise to the OOBI(s)
      */
     async get(name: string, role: string = 'agent'): Promise<any> {
-        const path = `/identifiers/${name}/oobis?role=${role}`;
+        const path = `/identifiers/${encodeURIComponent(name)}/oobis?role=${role}`;
         const method = 'GET';
         const res = await this.client.fetch(path, method, null);
         return await res.json();

diff --git a/src/keri/app/credentialing.ts b/src/keri/app/credentialing.ts
@@ -255,7 +255,7 @@ export class Credentials {
 
         const sigs = await keeper.sign(b(anc.raw));
 
-        const path = `/identifiers/${hab.name}/credentials`;
+        const path = `/identifiers/${encodeURIComponent(hab.name)}/credentials`;
         const method = 'POST';
         const body = {
             acdc: acdc,
@@ -363,7 +363,7 @@ export class Credentials {
             [keeper.algo]: keeper.params(),
         };
 
-        const path = `/identifiers/${name}/credentials/${said}`;
+        const path = `/identifiers/${encodeURIComponent(name)}/credentials/${said}`;
         const method = 'DELETE';
         const headers = new Headers({
             Accept: 'application/json+cesr',
@@ -433,7 +433,7 @@ export class Credentials {
             include: include,
         };
 
-        const path = `/identifiers/${name}/credentials/${said}/presentations`;
+        const path = `/identifiers/${encodeURIComponent(name)}/credentials/${said}/presentations`;
         const method = 'POST';
         const headers = new Headers({
             Accept: 'application/json+cesr',
@@ -496,7 +496,7 @@ export class Credentials {
             recipient: recipient,
         };
 
-        const path = `/identifiers/${name}/requests`;
+        const path = `/identifiers/${encodeURIComponent(name)}/requests`;
         const method = 'POST';
         const headers = new Headers({
             Accept: 'application/json+cesr',
@@ -571,7 +571,7 @@ export class Registries {
      * @returns {Promise<any>} A promise to the list of registries
      */
     async list(name: string): Promise<any> {
-        const path = `/identifiers/${name}/registries`;
+        const path = `/identifiers/${encodeURIComponent(name)}/registries`;
         const method = 'GET';
         const res = await this.client.fetch(path, method, null);
         return await res.json();
@@ -652,7 +652,7 @@ export class Registries {
         ixn: Dict<any>,
         sigs: any[]
     ) {
-        const path = `/identifiers/${name}/registries`;
+        const path = `/identifiers/${encodeURIComponent(name)}/registries`;
         const method = 'POST';
 
         const data: any = {
@@ -680,7 +680,7 @@ export class Registries {
         registryName: string,
         newName: string
     ): Promise<any> {
-        const path = `/identifiers/${name}/registries/${registryName}`;
+        const path = `/identifiers/${encodeURIComponent(name)}/registries/${registryName}`;
         const method = 'PUT';
         const data = {
             name: newName,
@@ -804,7 +804,7 @@ export class Ipex {
         };
 
         const response = await this.client.fetch(
-            `/identifiers/${name}/ipex/grant`,
+            `/identifiers/${encodeURIComponent(name)}/ipex/grant`,
             'POST',
             body
         );
@@ -860,7 +860,7 @@ export class Ipex {
         };
 
         const response = await this.client.fetch(
-            `/identifiers/${name}/ipex/admit`,
+            `/identifiers/${encodeURIComponent(name)}/ipex/admit`,
             'POST',
             body
         );

diff --git a/src/keri/app/exchanging.ts b/src/keri/app/exchanging.ts
@@ -113,7 +113,7 @@ export class Exchanges {
         atc: string,
         recipients: string[]
     ): Promise<any> {
-        const path = `/identifiers/${name}/exchanges`;
+        const path = `/identifiers/${encodeURIComponent(name)}/exchanges`;
         const method = 'POST';
         const data: any = {
             tpc: topic,

diff --git a/src/keri/app/grouping.ts b/src/keri/app/grouping.ts
@@ -43,7 +43,7 @@ export class Groups {
         sigs: string[],
         atc: string
     ): Promise<any> {
-        const path = `/identifiers/${name}/multisig/request`;
+        const path = `/identifiers/${encodeURIComponent(name)}/multisig/request`;
         const method = 'POST';
         const data = {
             exn: exn,
@@ -74,7 +74,7 @@ export class Groups {
         smids: string[],
         rmids: string[]
     ): Promise<any> {
-        const path = `/identifiers/${name}/multisig/join`;
+        const path = `/identifiers/${encodeURIComponent(name)}/multisig/join`;
         const method = 'POST';
         const data = {
             tpc: 'multisig',