darkwallet/darkwallet

Function receiveMessage has 95 lines of code (exceeds 25 allowed). Consider refactoring.

Open

multiParty.receiveMessage = function(sender, myName, message) {
    try {
        message = JSON.parse(message);
    }
    catch(err) {

File multiParty.js has 294 lines of code (exceeds 250 allowed). Consider refactoring.

Open

'use strict';

define(['bitcoinjs-lib', 'util/djbec', 'crypto-js'], function(Bitcoin, Curve25519, CryptoJS) {

var BigInteger = Bitcoin.BigInteger;

Function sendMessage has 27 lines of code (exceeds 25 allowed). Consider refactoring.

Open

multiParty.sendMessage = function(message) {
    //Convert from UTF8
    message = CryptoJS.enc.Utf8.parse(message);
    // Add 64 bytes of padding
    message.concat(getRawBytes(64));

Avoid deeply nested control flow statements.

Open

                    if (typeof(message['text'][recipients[r]]) === 'object') {
                        var noMessage = typeof(message['text'][recipients[r]]['message']) !== 'string';
                        var noIV = typeof(message['text'][recipients[r]]['iv']) !== 'string';
                        var noHMAC = typeof(message['text'][recipients[r]]['hmac']) !== 'string';
                        if (noMessage || noIV || noHMAC) {

Avoid too many return statements within this function.

Open

                return false;

Avoid too many return statements within this function.

Open

    return false;

Avoid too many return statements within this function.

Open

                return false;

Avoid too many return statements within this function.

Open

                return false;

Avoid too many return statements within this function.

Open

            return plaintext.toString(CryptoJS.enc.Utf8);

Avoid too many return statements within this function.

Open

                return false;

Identical blocks of code found in 2 locations. Consider refactoring.

Open

    sharedSecrets[user] = {
        'message': CryptoJS.lib.WordArray.create(sharedSecret.words.slice(0, 8)),
        'hmac': CryptoJS.lib.WordArray.create(sharedSecret.words.slice(8, 16))
    };

• Read up
  Read up

Identical blocks of code found in 2 locations. Consider refactoring.

Open

multiParty.messageTag = function(message) {
    for (var i = 0; i !== 8; i++) {
        message = CryptoJS.SHA512(message);
    }
    return message.toString(CryptoJS.enc.Base64);

• Read up
  Read up