File peers.py has 402 lines of code (exceeds 250 allowed). Consider refactoring.

Open

from multiprocessing import Queue
import multiprocessing, os, pickle, select, socket, sys, time, rsa, traceback
from common.safeprint import safeprint
from common.bounty import *

File bounty.py has 340 lines of code (exceeds 250 allowed). Consider refactoring.

Open

import os, pickle, re, sys, rsa
from common.safeprint import safeprint
from common.call import parse
from multiprocessing import Lock
from hashlib import sha256

Function listenp has a Cognitive Complexity of 18 (exceeds 5 allowed). Consider refactoring.

Open

def listenp(port, v):
    """BLOCKING function which should only be run in a daemon thread. Listens and responds to other nodes"""
    import time
    while v.value:   # is True is implicit
        safeprint("listenp-ing", verbosity=3)

• Read up
  Read up

Function listen has a Cognitive Complexity of 16 (exceeds 5 allowed). Consider refactoring.

Open

def listen(port, outbound, q, v, serv):
    """BLOCKING function which should only be run in a daemon thread. Listens and responds to other nodes"""
    if serv:
        from server.bounty import verify, addBounty
    server = socket.socket()

• Read up
  Read up

Function process has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

def process(tup):
    """Convert tuples into a format readable by call.call"""
    args = []
    ix = None
    ex = None

• Read up
  Read up

Function main has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

def main():
    # Begin Init
    settings.setup()
    try:
        import miniupnpc

• Read up
  Read up

Function call has a Cognitive Complexity of 9 (exceeds 5 allowed). Consider refactoring.

Open

def call(mod, cmd, *args, **kargs):
    """Calls arbitrary python code

    Arguments:
        mod     - The module from which you are calling

• Read up
  Read up

Function depickle has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def depickle(string):
    """Handles the potential errors in unpickling a bounty"""
    if isinstance(string, Bounty):
        return string
    safeprint([sys.version_info[0], sys.version_info[1], sys.version_info[2]])

• Read up
  Read up

Function isValid has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

    def isValid(self):
        """Internal method which checks the Bounty as valid in the most minimal version

        ip      -- Must be in valid range
        btc     -- Must be in valid namespace

• Read up
  Read up

Function verify has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def verify(string):
    """External method which checks the Bounty as valid under implementation-specific requirements. This can be defined per user.

    ip      -- Must be in valid range
    btc     -- Must be in valid namespace

• Read up
  Read up

Function handleBountyRequest has a Cognitive Complexity of 8 (exceeds 5 allowed). Consider refactoring.

Open

def handleBountyRequest(conn, exchange, key=None, received=[]):
    """Given a socket, send the proper messages to complete a bounty request"""
    unfiltered = getBountyList()
    filtered = list(set(unfiltered) - set(received))
    toSend = pickle.dumps(filtered, 0)

• Read up
  Read up

Function __init__ has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

    def __init__(self, port, outbound, q, v, serv):

Function listen has 5 arguments (exceeds 4 allowed). Consider refactoring.

Open

def listen(port, outbound, q, v, serv):

Function addBounties has a Cognitive Complexity of 7 (exceeds 5 allowed). Consider refactoring.

Open

def addBounties(bounties):
    """Add a list of bounties in parallel using multiprocessing.Pool for verification"""
    from multiprocessing.pool import ThreadPool
    pool = ThreadPool()
    safeprint("Mapping verifications", verbosity=3)

• Read up
  Read up

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 1

Avoid too many return statements within this function.

Open

        return -1

Avoid too many return statements within this function.

Open

            return -1

Avoid too many return statements within this function.

Open

                return 1