Skip to content

Commit

Permalink
Merge pull request #3626 from WalletConnect/chore/verify-log-level
Browse files Browse the repository at this point in the history
refactor: verify logs level
  • Loading branch information
ganchoradkov authored Sep 11, 2023
2 parents 0401bfc + 959c1b2 commit b36274a
Show file tree
Hide file tree
Showing 2 changed files with 7 additions and 7 deletions.
12 changes: 6 additions & 6 deletions packages/core/src/controllers/verify.ts
Original file line number Diff line number Diff line change
Expand Up @@ -42,8 +42,8 @@ export class Verify extends IVerify {
try {
await this.createIframe();
} catch (error) {
this.logger.warn(`Verify iframe failed to load: ${this.verifyUrl}`);
this.logger.warn(error);
this.logger.info(`Verify iframe failed to load: ${this.verifyUrl}`);
this.logger.info(error);
}

if (this.initialized) return;
Expand All @@ -54,8 +54,8 @@ export class Verify extends IVerify {
try {
await this.createIframe();
} catch (error) {
this.logger.error(`Verify iframe failed to load: ${this.verifyUrl}`);
this.logger.error(error);
this.logger.info(`Verify iframe failed to load: ${this.verifyUrl}`);
this.logger.info(error);
// if the fallback url fails to load as well, disable verify
this.verifyDisabled = true;
}
Expand All @@ -77,10 +77,10 @@ export class Verify extends IVerify {
try {
result = await this.fetchAttestation(params.attestationId, mainUrl);
} catch (error) {
this.logger.warn(
this.logger.info(
`failed to resolve attestation: ${params.attestationId} from url: ${mainUrl}`,
);
this.logger.warn(error);
this.logger.info(error);
result = await this.fetchAttestation(params.attestationId, VERIFY_FALLBACK_SERVER);
}
return result;
Expand Down
2 changes: 1 addition & 1 deletion packages/sign-client/src/controllers/engine.ts
Original file line number Diff line number Diff line change
Expand Up @@ -1455,7 +1455,7 @@ export class Engine extends IEngine {
context.verified.validation = origin === new URL(metadata.url).origin ? "VALID" : "INVALID";
}
} catch (e) {
this.client.logger.error(e);
this.client.logger.info(e);
}

this.client.logger.info(`Verify context: ${JSON.stringify(context)}`);
Expand Down

0 comments on commit b36274a

Please sign in to comment.