cyberark/secretless-broker

View on GitHub

Showing 1,856 of 1,876 total issues

TODO found
Open

    // TODO: the line number in this error is dishonest

TODO found
Open

    # TODO: Use `go vet` too
Severity: Minor
Found in bin/test_unit by fixme

TODO found
Open

    // TODO: Create a utility xprod function similar to the one here:

TODO found
Open

    //TODO: deal with verbose

TODO found
Open

    // TODO: we should verify with a channel that outer goroutine actually stops.

TODO found
Open

    // TODO: NewHTTPProxyFunc needs to be injected

TODO found
Open

// TODO: v1.Provider options should be an interface

TODO found
Open

TODO: Add a principled explanation about how logging is working.
Severity: Minor
Found in internal/proxyservice/doc.go by fixme

TODO found
Open

// TODO: consider parametrising ConnectPort generator

TODO found
Open

//TODO: Something is still quite wrong with the design here:
Severity: Minor
Found in test/util/testutil/types.go by fixme

TODO found
Open

    // TODO: If we want to unit test this, we'll need to inject net.Listen

TODO found
Open

    // TODO: v2.NetworkAddress is a value type.  It needs to be moved to its

TODO found
Open

    // TODO: Ideally we would soft-close all goroutines rather than rely on the

TODO found
Open

// TODO: The two methods below are currently NOT on the EventNotifier

Use of assert detected. The enclosed code will be removed when compiling to optimised byte code.
Open

assert sys.version_info >= (3, 0), "Python3 should be used to run this program"

Severity: Info
Found in bin/juxtaposer/aggregate.py by bandit

Refactor this function to reduce its Cognitive Complexity from 19 to the 15 allowed.
Open

def process_log(logfile_path, baseline_backend):
Severity: Critical
Found in bin/juxtaposer/aggregate.py by sonar-python

Cognitive Complexity is a measure of how hard the control flow of a function is to understand. Functions with high Cognitive Complexity will be difficult to maintain.

See

Severity
Category
Status
Source
Language