Browse Source

fix: resolve type compatibility in offerGiverDid

- Update offerGiverDid to use GenericVerifiableCredential as base type
- Add type assertion for OfferVerifiableCredential inside function
- Remove unnecessary type assertion in canFulfillOffer
pull/137/head
Matthew Raymer 2 weeks ago
parent
commit
068662625d
  1. 50
      src/libs/endorserServer.ts
  2. 13
      src/libs/util.ts

50
src/libs/endorserServer.ts

@ -110,11 +110,11 @@ export const BLANK_GENERIC_SERVER_RECORD: GenericCredWrapper<GenericVerifiableCr
"@context": SCHEMA_ORG_CONTEXT, "@context": SCHEMA_ORG_CONTEXT,
"@type": "" "@type": ""
}, },
handleId: "", handleId: "",
id: "", id: "",
issuedAt: "", issuedAt: "",
issuer: "", issuer: "",
}; };
// This is used to check for hidden info. // This is used to check for hidden info.
// See https://github.com/trentlarson/endorser-ch/blob/0cb626f803028e7d9c67f095858a9fc8542e3dbd/server/api/services/util.js#L6 // See https://github.com/trentlarson/endorser-ch/blob/0cb626f803028e7d9c67f095858a9fc8542e3dbd/server/api/services/util.js#L6
@ -209,8 +209,8 @@ const testRecursivelyOnStrings = (
return Object.values(input as Record<string, unknown>).some((value) => return Object.values(input as Record<string, unknown>).some((value) =>
testRecursivelyOnStrings(value, test) testRecursivelyOnStrings(value, test)
); );
} }
return false; return false;
}; };
// eslint-disable-next-line @typescript-eslint/no-explicit-any // eslint-disable-next-line @typescript-eslint/no-explicit-any
@ -582,17 +582,17 @@ export function errorStringForLog(error: unknown) {
if (error && typeof error === 'object' && 'response' in error) { if (error && typeof error === 'object' && 'response' in error) {
const err = error as AxiosErrorResponse; const err = error as AxiosErrorResponse;
const errorResponseText = JSON.stringify(err.response); const errorResponseText = JSON.stringify(err.response);
// for some reason, error.response is not included in stringify result (eg. for 400 errors on invite redemptions) // for some reason, error.response is not included in stringify result (eg. for 400 errors on invite redemptions)
if (!R.empty(errorResponseText) && !fullError.includes(errorResponseText)) { if (!R.empty(errorResponseText) && !fullError.includes(errorResponseText)) {
// add error.response stuff // add error.response stuff
if (err.response?.config && err.config && R.equals(err.config, err.response.config)) { if (err.response?.config && err.config && R.equals(err.config, err.response.config)) {
// but exclude "config" because it's already in there // but exclude "config" because it's already in there
const newErrorResponseText = JSON.stringify( const newErrorResponseText = JSON.stringify(
R.omit(["config"] as never[], err.response), R.omit(["config"] as never[], err.response),
); );
fullError += " - .response w/o same config JSON: " + newErrorResponseText; fullError += " - .response w/o same config JSON: " + newErrorResponseText;
} else { } else {
fullError += " - .response JSON: " + errorResponseText; fullError += " - .response JSON: " + errorResponseText;
} }
} }
} }
@ -1340,8 +1340,8 @@ export async function register(
contact: Contact, contact: Contact,
): Promise<{ success?: boolean; error?: string }> { ): Promise<{ success?: boolean; error?: string }> {
try { try {
const vcJwt = await createInviteJwt(activeDid, contact); const vcJwt = await createInviteJwt(activeDid, contact);
const url = apiServer + "/api/v2/claim"; const url = apiServer + "/api/v2/claim";
const resp = await axios.post<{ const resp = await axios.post<{
success?: { success?: {
handleId?: string; handleId?: string;
@ -1351,15 +1351,15 @@ export async function register(
message?: string; message?: string;
}>(url, { jwtEncoded: vcJwt }); }>(url, { jwtEncoded: vcJwt });
if (resp.data?.success?.handleId) { if (resp.data?.success?.handleId) {
return { success: true }; return { success: true };
} else if (resp.data?.success?.embeddedRecordError) { } else if (resp.data?.success?.embeddedRecordError) {
let message = "There was some problem with the registration and so it may not be complete."; let message = "There was some problem with the registration and so it may not be complete.";
if (typeof resp.data.success.embeddedRecordError === "string") { if (typeof resp.data.success.embeddedRecordError === "string") {
message += " " + resp.data.success.embeddedRecordError; message += " " + resp.data.success.embeddedRecordError;
} }
return { error: message }; return { error: message };
} else { } else {
logger.error("Registration error:", JSON.stringify(resp.data)); logger.error("Registration error:", JSON.stringify(resp.data));
return { error: "Got a server error when registering." }; return { error: "Got a server error when registering." };
} }

13
src/libs/util.ts

@ -384,14 +384,15 @@ export function base64ToBlob(base64DataUrl: string, sliceSize = 512) {
* @param veriClaim is expected to have fields: claim and issuer * @param veriClaim is expected to have fields: claim and issuer
*/ */
export function offerGiverDid( export function offerGiverDid(
veriClaim: GenericCredWrapper<OfferVerifiableCredential>, veriClaim: GenericCredWrapper<GenericVerifiableCredential>,
): string | undefined { ): string | undefined {
let giver; let giver;
const claim = veriClaim.claim as OfferVerifiableCredential;
if ( if (
veriClaim.claim.offeredBy?.identifier && claim.offeredBy?.identifier &&
!serverUtil.isHiddenDid(veriClaim.claim.offeredBy.identifier as string) !serverUtil.isHiddenDid(claim.offeredBy.identifier as string)
) { ) {
giver = veriClaim.claim.offeredBy.identifier; giver = claim.offeredBy.identifier;
} else if (veriClaim.issuer && !serverUtil.isHiddenDid(veriClaim.issuer)) { } else if (veriClaim.issuer && !serverUtil.isHiddenDid(veriClaim.issuer)) {
giver = veriClaim.issuer; giver = veriClaim.issuer;
} }
@ -407,7 +408,7 @@ export const canFulfillOffer = (
) => { ) => {
return ( return (
veriClaim.claimType === "Offer" && veriClaim.claimType === "Offer" &&
!!offerGiverDid(veriClaim as GenericCredWrapper<OfferVerifiableCredential>) !!offerGiverDid(veriClaim)
); );
}; };
@ -486,7 +487,7 @@ export const retrieveAccountCount = async (): Promise<number> => {
`SELECT COUNT(*) FROM accounts`, `SELECT COUNT(*) FROM accounts`,
); );
if (dbResult?.values?.[0]?.[0]) { if (dbResult?.values?.[0]?.[0]) {
result = dbResult.values[0][0] as number; result = dbResult.values[0][0] as number;
} }
if (USE_DEXIE_DB) { if (USE_DEXIE_DB) {

Loading…
Cancel
Save