Skip to content
This repository has been archived by the owner on May 24, 2022. It is now read-only.

Commit

Permalink
add comment explaining the issue
Browse files Browse the repository at this point in the history
  • Loading branch information
pmespresso authored Jan 2, 2019
1 parent a577e2b commit 71fbf5a
Showing 1 changed file with 3 additions and 0 deletions.
3 changes: 3 additions & 0 deletions packages/api/src/transport/ws/ws.js
Original file line number Diff line number Diff line change
Expand Up @@ -261,6 +261,9 @@ class Ws extends JsonRpcBase {

// Don't print error if request rejected or not is not yet up...
if (!/(rejected|not yet up)/.test(result.error.message)) {
// fether Issue #317
// js-libs Issue #77 Masks the password param when logging error to console on methods that contain it as a param.
// e.g. ["0x2",{},"myincorrectpassword"] -> ["0x2",{},"***"]
const dangerous_methods = ['signer_confirmRequest', 'signer_confirmRequestWithToken'];
let safe_params;
if (dangerous_methods.includes(method)) {
Expand Down

0 comments on commit 71fbf5a

Please sign in to comment.