{"$schema":"https://json.schemastore.org/sarif-2.1.0.json","version":"2.1.0","runs":[{"tool":{"driver":{"name":"codehealth","informationUri":"https://codehealth.canine.dev","rules":[{"id":"D1","name":"Cyclomatic Complexity","shortDescription":{"text":"Cyclomatic Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D1"},{"id":"D2","name":"Cognitive Complexity","shortDescription":{"text":"Cognitive Complexity"},"helpUri":"https://codehealth.canine.dev/dimensions/D2"},{"id":"D3","name":"God Classes","shortDescription":{"text":"God Classes"},"helpUri":"https://codehealth.canine.dev/dimensions/D3"},{"id":"D4","name":"Code Duplication","shortDescription":{"text":"Code Duplication"},"helpUri":"https://codehealth.canine.dev/dimensions/D4"},{"id":"D13","name":"Secret Scanning","shortDescription":{"text":"Secret Scanning"},"helpUri":"https://codehealth.canine.dev/dimensions/D13","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D15","name":"Churn \u00D7 Complexity Hotspots","shortDescription":{"text":"Churn \u00D7 Complexity Hotspots"},"helpUri":"https://codehealth.canine.dev/dimensions/D15"},{"id":"D16","name":"Bus Factor","shortDescription":{"text":"Bus Factor"},"helpUri":"https://codehealth.canine.dev/dimensions/D16"},{"id":"D19","name":"Documentation Quality","shortDescription":{"text":"Documentation Quality"},"helpUri":"https://codehealth.canine.dev/dimensions/D19"},{"id":"D21","name":"Naming Consistency","shortDescription":{"text":"Naming Consistency"},"helpUri":"https://codehealth.canine.dev/dimensions/D21"},{"id":"D28","name":"Secrets (history)","shortDescription":{"text":"Secrets (history)"},"helpUri":"https://codehealth.canine.dev/dimensions/D28","relationships":[{"target":{"id":"CWE-798","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-259","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-798","CWE-259"]}},{"id":"D29","name":"Static Analysis (SAST)","shortDescription":{"text":"Static Analysis (SAST)"},"helpUri":"https://codehealth.canine.dev/dimensions/D29","relationships":[{"target":{"id":"CWE-79","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-89","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-78","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-94","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-77","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-79","CWE-89","CWE-78","CWE-94","CWE-77"]}},{"id":"D33","name":"JS/npm Dependency Vulnerabilities","shortDescription":{"text":"JS/npm Dependency Vulnerabilities"},"helpUri":"https://codehealth.canine.dev/dimensions/D33","relationships":[{"target":{"id":"CWE-1395","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-937","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1395","CWE-937"]}},{"id":"D34","name":"Knowledge Freshness","shortDescription":{"text":"Knowledge Freshness"},"helpUri":"https://codehealth.canine.dev/dimensions/D34"},{"id":"D35","name":"Change Coupling","shortDescription":{"text":"Change Coupling"},"helpUri":"https://codehealth.canine.dev/dimensions/D35"},{"id":"D36","name":"Supply-chain Provenance \u0026 Signing","shortDescription":{"text":"Supply-chain Provenance \u0026 Signing"},"helpUri":"https://codehealth.canine.dev/dimensions/D36","relationships":[{"target":{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-494","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1357","CWE-494"]}},{"id":"D38","name":"OSV Dependency Vulnerabilities","shortDescription":{"text":"OSV Dependency Vulnerabilities"},"helpUri":"https://codehealth.canine.dev/dimensions/D38","relationships":[{"target":{"id":"CWE-1395","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]},{"target":{"id":"CWE-937","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},"kinds":["relevant"]}],"properties":{"cwe":["CWE-1395","CWE-937"]}}]}},"results":[{"ruleId":"D1","level":"warning","message":{"text":"server.NewService (cyclomatic 30): server.NewService has cyclomatic complexity 30 (threshold 15). Of this number, 29 points are the body\u0027s own statements and 1 belongs to one function literal inside it that branches. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/service.go"},"region":{"startLine":138}}}],"partialFingerprints":{"codehealthFindingId/v1":"bc43b22f92110cf739e30c3634e054e69938ced5359d939b067a03702dcc1581"}},{"ruleId":"D1","level":"warning","message":{"text":"UDPProxy.Run (cyclomatic 27): UDPProxy.Run has cyclomatic complexity 27 (threshold 15). Most of this is not in the body itself: 4 of the 27 points are its own statements and the rest belongs to 4 function literals inside it that branch (lines 184, 111, 156, \u2026). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/proxy/udp.go"},"region":{"startLine":78}}}],"partialFingerprints":{"codehealthFindingId/v1":"8125b198e84094fba738367d9900bfee36388e40dc01e182760ec286dad94a64"}},{"ruleId":"D1","level":"warning","message":{"text":"nathole.MakeHole (cyclomatic 23): nathole.MakeHole has cyclomatic complexity 23 (threshold 15). Of this number, 21 points are the body\u0027s own statements and 2 belong to one function literal inside it that branches. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/nathole/nathole.go"},"region":{"startLine":192}}}],"partialFingerprints":{"codehealthFindingId/v1":"44ca0ad8ecc4ae96085f70d33630762b6814b445137219e909837a1702129a22"}},{"ruleId":"D1","level":"warning","message":{"text":"legacy.LoadAllProxyConfsFromIni (cyclomatic 19): legacy.LoadAllProxyConfsFromIni has cyclomatic complexity 19 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/legacy/client.go"},"region":{"startLine":204}}}],"partialFingerprints":{"codehealthFindingId/v1":"409d4ddc0450d385e654d453c81f9d710aea99de618e907fb8e8a6edb71fb01c"}},{"ruleId":"D1","level":"warning","message":{"text":"SUDPProxy.InWorkConn (cyclomatic 17): SUDPProxy.InWorkConn has cyclomatic complexity 17 (threshold 15). Most of this is not in the body itself: 3 of the 17 points are its own statements and the rest belongs to 4 function literals inside it that branch (lines 119, 147, 170, \u2026). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/proxy/sudp.go"},"region":{"startLine":79}}}],"partialFingerprints":{"codehealthFindingId/v1":"f4b307de26114122357221abc7463a7e9f7af2bd5c69cb1adaaf9e21c07527be"}},{"ruleId":"D1","level":"warning","message":{"text":"http.buildV2ProxySpec (cyclomatic 17): http.buildV2ProxySpec has cyclomatic complexity 17 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/http/controller_v2.go"},"region":{"startLine":564}}}],"partialFingerprints":{"codehealthFindingId/v1":"f0f5dafea613e0ceab3b8358222e5911263171e20e0ead643d42455d675730df"}},{"ruleId":"D1","level":"warning","message":{"text":"controlSessionDialer.exchangeLogin (cyclomatic 16): controlSessionDialer.exchangeLogin has cyclomatic complexity 16 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/control_session.go"},"region":{"startLine":136}}}],"partialFingerprints":{"codehealthFindingId/v1":"2a872cbe0cdacd49af29fe6ae0b0a533e02f36fff7b92c2dc2e79d28094fccd8"}},{"ruleId":"D1","level":"warning","message":{"text":"nathole.ClassifyNATFeature (cyclomatic 16): nathole.ClassifyNATFeature has cyclomatic complexity 16 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/nathole/classify.go"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"c1f18def1c6bf3231a1ab8eba97dffe1f4539664465a6bcf57a392a0f9f4e169"}},{"ruleId":"D1","level":"warning","message":{"text":"Controller.APIV2ClientList (cyclomatic 16): Controller.APIV2ClientList has cyclomatic complexity 16 (threshold 15). Of this number, 14 points are the body\u0027s own statements and 2 belong to one function literal inside it that branches. To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/http/controller_v2.go"},"region":{"startLine":143}}}],"partialFingerprints":{"codehealthFindingId/v1":"6bd68977377b0ac3788010d585c2c34366cbb9a2196798f539eca3fddf5845d2"}},{"ruleId":"D1","level":"warning","message":{"text":"ClientRegistry.RegisterWithControlID (cyclomatic 16): ClientRegistry.RegisterWithControlID has cyclomatic complexity 16 (threshold 15). To reduce it, split the body: these branches sit side by side rather than nested inside one another, so extracting each one on its own would leave a function per branch. Group the statements between the checks into named steps and move each step into its own function, so the body reads as a short sequence of named stages."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/registry/registry.go"},"region":{"startLine":74}}}],"partialFingerprints":{"codehealthFindingId/v1":"e9d1ba9ddea50b683d21304ffabe2875902003a1d79c825bda02451a8ca1fba3"}},{"ruleId":"D1","level":"warning","message":{"text":"Service.handleConnection (cyclomatic 16): Service.handleConnection has cyclomatic complexity 16 (threshold 15). To reduce it, separate the cases: extract each independent branch into its own named function, and where the body has guards that only reject input, fold those into early returns at the top."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/service.go"},"region":{"startLine":442}}}],"partialFingerprints":{"codehealthFindingId/v1":"858d02a564da75eeb0c1a7a4d4ff6950fb66a271114a4eccf6de38721d908151"}},{"ruleId":"D2","level":"warning","message":{"text":"UDPProxy.Run (cognitive 48): UDPProxy.Run has cognitive complexity 48 (threshold 15). Drivers by points: if/else 38, match/switch 7, loops 3 (nesting depth added 24). Most of this is not in the body itself: 3 of the 48 points are its own statements and the rest belongs to 4 function literals inside it that branch (lines 184, 111, 156, \u2026). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/proxy/udp.go"},"region":{"startLine":78}}}],"partialFingerprints":{"codehealthFindingId/v1":"644d2d7e4b465d725753addbf0dd58312eec17a1021a7c8f7c47c6497a958553"}},{"ruleId":"D2","level":"warning","message":{"text":"server.NewService (cognitive 47): server.NewService has cognitive complexity 47 (threshold 15). Drivers by points: if/else 45, boolean chains 1, loops 1 (nesting depth added 16). Of this number, 46 points are the body\u0027s own statements and 1 belongs to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/service.go"},"region":{"startLine":138}}}],"partialFingerprints":{"codehealthFindingId/v1":"64403d60d506f6e9f7d08e1ffa11f198de11a2226d3bf3095b74ad9498874f4f"}},{"ruleId":"D2","level":"warning","message":{"text":"nathole.MakeHole (cognitive 37): nathole.MakeHole has cognitive complexity 37 (threshold 15). Drivers by points: if/else 23, loops 11, match/switch 3 (nesting depth added 16). Of this number, 33 points are the body\u0027s own statements and 4 belong to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/nathole/nathole.go"},"region":{"startLine":192}}}],"partialFingerprints":{"codehealthFindingId/v1":"c7392591d093ffb34df9576596ca5a7dd5001b02ce726752b325f8e45152e0c8"}},{"ruleId":"D2","level":"warning","message":{"text":"Service.handleConnection (cognitive 33): Service.handleConnection has cognitive complexity 33 (threshold 15). Drivers by points: if/else 32, match/switch 1 (nesting depth added 18). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/service.go"},"region":{"startLine":442}}}],"partialFingerprints":{"codehealthFindingId/v1":"184270757832f8dd68547d759739570f4c7e2cadea62bc018dc57820bc259772"}},{"ruleId":"D2","level":"warning","message":{"text":"Manager.Acquire (cognitive 30): Manager.Acquire has cognitive complexity 30 (threshold 15). Drivers by points: if/else 28, loops 2 (nesting depth added 14). Of this number, 29 points are the body\u0027s own statements and 1 belongs to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/ports/ports.go"},"region":{"startLine":82}}}],"partialFingerprints":{"codehealthFindingId/v1":"105c8b863704983b9ecf6c361de2dd30001051476f0f3460efbfd998d20355c0"}},{"ruleId":"D2","level":"warning","message":{"text":"config.LoadAdditionalClientConfigs (cognitive 27): config.LoadAdditionalClientConfigs has cognitive complexity 27 (threshold 15). Drivers by points: if/else 16, loops 11 (nesting depth added 17). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/load.go"},"region":{"startLine":502}}}],"partialFingerprints":{"codehealthFindingId/v1":"69f24a1eb8f4c41726b0702b4549b16fb8bab8b38c2eec14d61f885cc3d66845"}},{"ruleId":"D2","level":"warning","message":{"text":"legacy.LoadAllProxyConfsFromIni (cognitive 26): legacy.LoadAllProxyConfsFromIni has cognitive complexity 26 (threshold 15). Drivers by points: if/else 18, loops 4, boolean chains 2, match/switch 2 (nesting depth added 10). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/legacy/client.go"},"region":{"startLine":204}}}],"partialFingerprints":{"codehealthFindingId/v1":"9ebcc7dd7655a5d8ca28bbbecf01765bdae71c8340ec8048c76af6419ed21ab5"}},{"ruleId":"D2","level":"warning","message":{"text":"udp.Forwarder (cognitive 26): udp.Forwarder has cognitive complexity 26 (threshold 15). Drivers by points: if/else 21, loops 2, match/switch 2, boolean chains 1 (nesting depth added 12). Most of this is not in the body itself: 0 of the 26 points are its own statements and the rest belongs to 2 function literals inside it that branch (lines 109, 78). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/proto/udp/udp.go"},"region":{"startLine":73}}}],"partialFingerprints":{"codehealthFindingId/v1":"9b571a7c41e5dca8cd8e127aeaa9c68222f338d8a8f04156c1cf4d196f460be9"}},{"ruleId":"D2","level":"warning","message":{"text":"ControlManager.Add (cognitive 26): ControlManager.Add has cognitive complexity 26 (threshold 15). Drivers by points: if/else 25, loops 1 (nesting depth added 14). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/control.go"},"region":{"startLine":109}}}],"partialFingerprints":{"codehealthFindingId/v1":"c7594cb84e7e9678ccc53e3234f142590c6366f0cc395d028bf33e8e243f02c8"}},{"ruleId":"D2","level":"warning","message":{"text":"controlSessionDialer.exchangeLogin (cognitive 23): controlSessionDialer.exchangeLogin has cognitive complexity 23 (threshold 15). Drivers by points: if/else 22, boolean chains 1 (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/control_session.go"},"region":{"startLine":136}}}],"partialFingerprints":{"codehealthFindingId/v1":"44bba160ba8228123465388da2f0982a7f6582134fd4a4a14109b7cdd1374864"}},{"ruleId":"D2","level":"warning","message":{"text":"Controller.readLoopServer (cognitive 23): Controller.readLoopServer has cognitive complexity 23 (threshold 15). Drivers by points: if/else 21, boolean chains 1, loops 1 (nesting depth added 12). Of this number, 22 points are the body\u0027s own statements and 1 belongs to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/vnet/controller.go"},"region":{"startLine":193}}}],"partialFingerprints":{"codehealthFindingId/v1":"3cb28fc5dc750686ef0311f4ad84b0c0a75e62ae7dfd14ec60ffbe944b69818d"}},{"ruleId":"D2","level":"warning","message":{"text":"SUDPProxy.InWorkConn (cognitive 22): SUDPProxy.InWorkConn has cognitive complexity 22 (threshold 15). Drivers by points: if/else 13, match/switch 6, loops 3 (nesting depth added 8). Most of this is not in the body itself: 2 of the 22 points are its own statements and the rest belongs to 4 function literals inside it that branch (lines 119, 170, 147, \u2026). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/proxy/sudp.go"},"region":{"startLine":79}}}],"partialFingerprints":{"codehealthFindingId/v1":"cadb1526a2e4fbbc31f7d513aed30255a905c9bf323f095f54bdd701c9f71d9f"}},{"ruleId":"D2","level":"warning","message":{"text":"config.LoadClientConfigResult (cognitive 22): config.LoadClientConfigResult has cognitive complexity 22 (threshold 15). Drivers by points: if/else 13, loops 8, boolean chains 1 (nesting depth added 8). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/load.go"},"region":{"startLine":346}}}],"partialFingerprints":{"codehealthFindingId/v1":"d664d1307a1d21d191bed6ed086dedec00b43e7c094dcaecdad0829881373c60"}},{"ruleId":"D2","level":"warning","message":{"text":"BaseProxy.startCommonTCPListenersHandler (cognitive 22): BaseProxy.startCommonTCPListenersHandler has cognitive complexity 22 (threshold 15). Drivers by points: if/else 18, loops 3, boolean chains 1 (nesting depth added 14). Most of this is not in the body itself: 1 of the 22 points is its own statement and the rest belongs to one function literal inside it that branches (line 235). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/proxy/proxy.go"},"region":{"startLine":232}}}],"partialFingerprints":{"codehealthFindingId/v1":"a739a3b1060ec548e4d28c6dda4a20333caaf7946a317cab3d5d08d48f4d01a5"}},{"ruleId":"D2","level":"warning","message":{"text":"Service.HandleListener (cognitive 22): Service.HandleListener has cognitive complexity 22 (threshold 15). Drivers by points: if/else 17, loops 4, boolean chains 1 (nesting depth added 12). Most of this is not in the body itself: 8 of the 22 points are its own statements and the rest belongs to one function literal inside it that branches (line 713). The decisions are inside the literal, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literal\u0027s work into a named function or method at the enclosing scope and have the literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/service.go"},"region":{"startLine":684}}}],"partialFingerprints":{"codehealthFindingId/v1":"ec9311e6fe7d8e0703e4b988315d7ba520e08b2d435cc1039cccce3c29ea0093"}},{"ruleId":"D2","level":"warning","message":{"text":"SUDPVisitor.worker (cognitive 21): SUDPVisitor.worker has cognitive complexity 21 (threshold 15). Drivers by points: if/else 15, match/switch 4, loops 2 (nesting depth added 10). Most of this is not in the body itself: 1 of the 21 points is its own statement and the rest belongs to 2 function literals inside it that branch (lines 167, 129). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/visitor/sudp.go"},"region":{"startLine":113}}}],"partialFingerprints":{"codehealthFindingId/v1":"d5f8b9dbcd1bdf3f30b64b842db5f2ca59b9ba30398e2232d2de2f0da4dcc35e"}},{"ruleId":"D2","level":"warning","message":{"text":"nathole.ClassifyNATFeature (cognitive 20): nathole.ClassifyNATFeature has cognitive complexity 20 (threshold 15). Drivers by points: if/else 16, boolean chains 2, loops 1, match/switch 1 (nesting depth added 7). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/nathole/classify.go"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"89ffbf2002f03bee817485ae8838de9e12f12f9365191ae5ccedb8815ce9f18d"}},{"ruleId":"D2","level":"warning","message":{"text":"fastBackoffImpl.Backoff (cognitive 20): fastBackoffImpl.Backoff has cognitive complexity 20 (threshold 15). Drivers by points: if/else 18, boolean chains 2 (nesting depth added 6). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/util/wait/backoff.go"},"region":{"startLine":77}}}],"partialFingerprints":{"codehealthFindingId/v1":"151ff5542ae9061eeb3b0b865a3ccff4a0416c761fec69a6b4161100435623e1"}},{"ruleId":"D2","level":"warning","message":{"text":"Controller.APIV2ClientList (cognitive 20): Controller.APIV2ClientList has cognitive complexity 20 (threshold 15). Drivers by points: if/else 15, boolean chains 4, loops 1 (nesting depth added 5). Of this number, 18 points are the body\u0027s own statements and 2 belong to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/http/controller_v2.go"},"region":{"startLine":143}}}],"partialFingerprints":{"codehealthFindingId/v1":"0fa9a2f329f91493e898ff4b649f86ab52b0b98ebf1de79bb17b3aa707419438"}},{"ruleId":"D2","level":"warning","message":{"text":"legacy.getIncludeContents (cognitive 19): legacy.getIncludeContents has cognitive complexity 19 (threshold 15). Drivers by points: if/else 16, loops 3 (nesting depth added 11). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/legacy/parse.go"},"region":{"startLine":68}}}],"partialFingerprints":{"codehealthFindingId/v1":"ecd9d354c2f7fbe1d5449877ebf15a7bc839cce9133bbfac8fc0e018098d5ee8"}},{"ruleId":"D2","level":"warning","message":{"text":"util.ParseRangeNumbers (cognitive 18): util.ParseRangeNumbers has cognitive complexity 18 (threshold 15). Drivers by points: if/else 12, loops 4, match/switch 2 (nesting depth added 11). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/util/util/util.go"},"region":{"startLine":67}}}],"partialFingerprints":{"codehealthFindingId/v1":"4804c005b6e3959cfebeffc24f7760ebb9cd27fa00792729957fb0e1de467a72"}},{"ruleId":"D2","level":"warning","message":{"text":"Wrapper.checkWorker (cognitive 17): Wrapper.checkWorker has cognitive complexity 17 (threshold 15). Drivers by points: if/else 10, boolean chains 4, match/switch 2, loops 1 (nesting depth added 6). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/proxy/proxy_wrapper.go"},"region":{"startLine":195}}}],"partialFingerprints":{"codehealthFindingId/v1":"c5b26ad7b89247144ad8dbc07ccdb284fabba22849d7dfe332b2d0ce259696d0"}},{"ruleId":"D2","level":"warning","message":{"text":"ClientCommonConf.Validate (cognitive 17): ClientCommonConf.Validate has cognitive complexity 17 (threshold 15). Drivers by points: if/else 15, boolean chains 1, loops 1 (nesting depth added 6). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/legacy/client.go"},"region":{"startLine":362}}}],"partialFingerprints":{"codehealthFindingId/v1":"77d4c3bf34b5ed2c7d06c2285e09f0c81466cc11e137223206a418154c45e3ac"}},{"ruleId":"D2","level":"warning","message":{"text":"VirtualNetPlugin.run (cognitive 17): VirtualNetPlugin.run has cognitive complexity 17 (threshold 15). Drivers by points: if/else 9, match/switch 7, loops 1 (nesting depth added 9). To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/plugin/visitor/virtual_net.go"},"region":{"startLine":101}}}],"partialFingerprints":{"codehealthFindingId/v1":"927d6d943c14b93ea71814680e51b3ce6d3222e044660c05d88f411780e7130d"}},{"ruleId":"D2","level":"warning","message":{"text":"vhost.NewHTTPReverseProxy (cognitive 17): vhost.NewHTTPReverseProxy has cognitive complexity 17 (threshold 15). Drivers by points: if/else 12, loops 4, boolean chains 1 (nesting depth added 5). Most of this is not in the body itself: 1 of the 17 points is its own statement and the rest belongs to 4 function literals inside it that branch (lines 59, 128, 94, \u2026). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/util/vhost/http.go"},"region":{"startLine":49}}}],"partialFingerprints":{"codehealthFindingId/v1":"5fc13117ac01562c1a746a28e8de51aeab63189f482df016561e88ace816564c"}},{"ruleId":"D2","level":"warning","message":{"text":"http.buildV2ProxySpec (cognitive 17): http.buildV2ProxySpec has cognitive complexity 17 (threshold 15). Drivers by points: if/else 16, match/switch 1 (nesting depth added 8). The drivers above price the dispatch low by construction \u2014 a dispatch is charged once however many cases it lists, while each branch inside an arm is charged in full \u2014 so most of this count is what the case bodies hold, and the arms are where it can be reduced. To reduce it, keep the dispatch but shrink the arms: move each non-trivial case body into its own named function (or onto the value being matched) so the dispatch reads one line per case, and group related cases into a sub-dispatch. Keep every case explicit, and make the behaviour for cases you do not list a deliberate choice rather than an accident."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/http/controller_v2.go"},"region":{"startLine":564}}}],"partialFingerprints":{"codehealthFindingId/v1":"6da76adb9a812037f97b3d9e4dca8516ef903857c2c9bbf4a8934e25db9f65e0"}},{"ruleId":"D2","level":"warning","message":{"text":"HTTPProxy.Run (cognitive 17): HTTPProxy.Run has cognitive complexity 17 (threshold 15). Drivers by points: if/else 14, loops 3 (nesting depth added 9). Of this number, 16 points are the body\u0027s own statements and 1 belongs to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/proxy/http.go"},"region":{"startLine":55}}}],"partialFingerprints":{"codehealthFindingId/v1":"ec2485b0b53c612236c4a5046c1042445db694c0daddc3104217edfd8df790ac"}},{"ruleId":"D2","level":"warning","message":{"text":"TunnelServer.waitForwardAddrAndExtraPayload (cognitive 16): TunnelServer.waitForwardAddrAndExtraPayload has cognitive complexity 16 (threshold 15). Drivers by points: if/else 9, loops 3, boolean chains 2, match/switch 2 (nesting depth added 6). Most of this is not in the body itself: 6 of the 16 points are its own statements and the rest belongs to 2 function literals inside it that branch (lines 209, 227). The decisions are inside those literals, which nothing outside this body can call, review or test on its own, so splitting the enclosing body is not the move available here. To reduce it, lift the literals\u0027 work into a named function or method at the enclosing scope and have each literal call it, then reduce whichever part then reads as the largest."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/ssh/server.go"},"region":{"startLine":200}}}],"partialFingerprints":{"codehealthFindingId/v1":"b6fd9fa82f24c07202d20944076fc6a52b0c5b952260cce59cfc0a66cd2e1bf0"}},{"ruleId":"D2","level":"warning","message":{"text":"Controller.APIV2ProxyList (cognitive 16): Controller.APIV2ProxyList has cognitive complexity 16 (threshold 15). Drivers by points: if/else 12, boolean chains 3, loops 1 (nesting depth added 4). Of this number, 15 points are the body\u0027s own statements and 1 belongs to one function literal inside it that branches. To reduce it, split the body into named stages: move each independent step or branch into its own named function so the body reads as a short sequence of named calls rather than one long body."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/http/controller_v2.go"},"region":{"startLine":221}}}],"partialFingerprints":{"codehealthFindingId/v1":"7280f43e2d8b046bf2a3e4d88baa9a30a3f4e88b97d6a4782e8ca543de72676b"}},{"ruleId":"D2","level":"warning","message":{"text":"ClientRegistry.RegisterWithControlID (cognitive 16): ClientRegistry.RegisterWithControlID has cognitive complexity 16 (threshold 15). Drivers by points: if/else 13, boolean chains 3 (nesting depth added 3). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition, and where an else follows a branch that already returns, drop the trailing else and let the rest of the body continue at one level."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/registry/registry.go"},"region":{"startLine":74}}}],"partialFingerprints":{"codehealthFindingId/v1":"913412b935562c7f74fca137aad088a4e526acdfc37dea7a584db76ddf954ffd"}},{"ruleId":"D2","level":"warning","message":{"text":"Service.RegisterControl (cognitive 16): Service.RegisterControl has cognitive complexity 16 (threshold 15). Drivers by points: if/else 13, boolean chains 2, match/switch 1 (nesting depth added 3). To reduce it, split the body: most of this score is breadth rather than depth \u2014 checks laid out side by side rather than stacked \u2014 so group the statements between the checks into named steps and move each step into its own function. Some of it IS depth: where a check sits inside another whose only job is to reach it, merge the two into one condition."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/service.go"},"region":{"startLine":766}}}],"partialFingerprints":{"codehealthFindingId/v1":"0da082bfa76a32dd7084455bebb1ecc33ba52fd62bee2cc43e6fb8aa06f87ac8"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: server/service.go: FileTooLong \u2014 595 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/service.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"aa1c7b805a083c7140a29cca731e8df96e221f7883ff39c5c803356d724c5721"}},{"ruleId":"D3","level":"warning","message":{"text":"FileTooLong: server/control.go: FileTooLong \u2014 588 significant lines (blank, comment-only and punctuation-only lines excluded). To reduce it, split the file along the responsibilities already in it: move each cohesive group of declarations into its own sibling file in the same module or package, so no one file has to be read whole to change one of them."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/control.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"6d9095d560fb275d06e799b0423783e34a79344bfe9c597479a3c5d421597bf6"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (16 lines \u00D7 2): client/http/model/proxy_definition.go:122-138 | client/http/model/visitor_definition.go:82-97 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/http/model/proxy_definition.go"},"region":{"startLine":122}}}],"partialFingerprints":{"codehealthFindingId/v1":"e0104c6d9a6867c0060295e455183f6ddb4edf6e8703251d31eb53ce10320cc0"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (13 lines \u00D7 2): client/proxy/xtcp.go:63-75 | client/visitor/xtcp.go:258-270 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060client/proxy/xtcp.go:63\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/proxy/xtcp.go"},"region":{"startLine":63}}}],"partialFingerprints":{"codehealthFindingId/v1":"501f5a9a924fd12d3f8e83bac94aca3dcecb269255a38e00f17b3598020dc3c9"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (12 lines \u00D7 2): server/proxy/proxy.go:461-472 | server/proxy/proxy.go:485-496 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060server/proxy/proxy.go:461\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/proxy/proxy.go"},"region":{"startLine":461}}}],"partialFingerprints":{"codehealthFindingId/v1":"7e933cb8252d1fb8939f78ae7a85dd043deb0278828956c58ba33cf5d4e89bec"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): client/config_manager.go:395-405 | client/config_manager.go:421-431 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060client/config_manager.go:395\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also register a scope-exit action (a \u0060defer\u0060-style statement) that runs when the function holding them returns: moved into a called unit it would run when THAT unit returns instead \u2014 before the caller uses what it releases \u2014 so keep the registration at the call site and extract only the work around it, or have the extracted unit hand the resource back for the caller to register."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/config_manager.go"},"region":{"startLine":395}}}],"partialFingerprints":{"codehealthFindingId/v1":"4dcf2852e85f6ef12f3f4025725e38b2433ed5474e7abe776355f35930635ac2"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (11 lines \u00D7 2): cmd/frpc/sub/proxy.go:79-90 | cmd/frpc/sub/proxy.go:112-122 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060cmd/frpc/sub/proxy.go:79\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cmd/frpc/sub/proxy.go"},"region":{"startLine":79}}}],"partialFingerprints":{"codehealthFindingId/v1":"b3961df87d7be1099c9f1e05f636a1c990073e57a7a39ae1e188d576616f624c"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (10 lines \u00D7 2): pkg/config/types/types.go:140-149 | pkg/util/util/util.go:71-80 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/config/types/types.go:140\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/types/types.go"},"region":{"startLine":140}}}],"partialFingerprints":{"codehealthFindingId/v1":"6c26d34789f035d6d4df72f3336cead557e6e72db4569af9b28e2d27f9ac37e5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): client/proxy/sudp.go:84-92 | client/proxy/udp.go:95-104 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060client/proxy/sudp.go:84\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/proxy/sudp.go"},"region":{"startLine":84}}}],"partialFingerprints":{"codehealthFindingId/v1":"69e59d6b7d87e7db942a6d668bde069edff2eaee23baf90343730a41a64cd25a"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): pkg/config/legacy/value.go:27-35 | pkg/config/load.go:44-52 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/legacy/value.go"},"region":{"startLine":27}}}],"partialFingerprints":{"codehealthFindingId/v1":"688ed3d0cff3ab00a2164c778afbc1c0aac9e4af877c661916ee2a683c8d6a50"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): pkg/sdk/client/client.go:36-44 | pkg/sdk/client/client.go:59-67 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/sdk/client/client.go"},"region":{"startLine":36}}}],"partialFingerprints":{"codehealthFindingId/v1":"957d7d3550591cf60bd090472299c7992ffc77e004ec33d3179f6eb5072d55e7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): pkg/util/http/handler.go:42-50 | pkg/util/http/handler.go:78-86 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/util/http/handler.go:42\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/util/http/handler.go"},"region":{"startLine":42}}}],"partialFingerprints":{"codehealthFindingId/v1":"8c079cfb50542152050f03b3a49a33573dab249ffbd7ac91b9c656ae373539ca"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (9 lines \u00D7 2): server/http/controller.go:120-128 | server/http/controller_v2.go:185-193 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Read the line range as the matched WINDOW rather than a finished unit: at \u0060server/http/controller.go:120\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/http/controller.go"},"region":{"startLine":120}}}],"partialFingerprints":{"codehealthFindingId/v1":"aea9d3e2045abebdf5176a4d4af9b047421ca5840b8c67970c516a0ef36dc7e7"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): client/http/controller.go:243-250 | client/http/controller.go:278-285 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060client/http/controller.go:243\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/http/controller.go"},"region":{"startLine":243}}}],"partialFingerprints":{"codehealthFindingId/v1":"24df92ae33e7996f057276672c49af098ef0b9220daeb6f7238a295c165b934e"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): client/http/controller.go:359-366 | client/http/controller.go:394-401 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060client/http/controller.go:359\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/http/controller.go"},"region":{"startLine":359}}}],"partialFingerprints":{"codehealthFindingId/v1":"2d177f5b7d65fe8a749c7d1d6611d86bab471da78bc214df018db264ab44aac6"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): pkg/auth/auth.go:43-50 | pkg/auth/auth.go:101-108 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/auth/auth.go:43\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also transfer control out of the body holding them, which cannot survive a move into a called unit unchanged: have the extracted unit return that decision and let each site act on it."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/auth/auth.go"},"region":{"startLine":43}}}],"partialFingerprints":{"codehealthFindingId/v1":"b40a31b3e8c0e5d149c98ce243be2682f4e0d762349a3888d856288dcdb06ace"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (8 lines \u00D7 2): pkg/config/legacy/conversion.go:30-37 | pkg/config/legacy/conversion.go:91-98 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/legacy/conversion.go"},"region":{"startLine":30}}}],"partialFingerprints":{"codehealthFindingId/v1":"2913d0b8b07873381b0088978d90bd9c6df27c2b09791bfe04eb3c465c6ab14f"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): client/http/controller.go:285-291 | client/http/controller.go:401-407 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060client/http/controller.go:285\u0060 it begins part-way through the construct above it, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/http/controller.go"},"region":{"startLine":285}}}],"partialFingerprints":{"codehealthFindingId/v1":"0dc627007e1e23ef4160d24ffe9ff4a741576232e56fbae85bc6d199c2e789fe"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): pkg/config/load.go:517-524 | pkg/config/legacy/parse.go:82-88 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060pkg/config/load.go:517\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/load.go"},"region":{"startLine":517}}}],"partialFingerprints":{"codehealthFindingId/v1":"d5ab042a3c9f5dabdd3f83007414fe3fb782bf06739ea2be9813b97ad7c28cd5"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): pkg/msg/udp_binary.go:304-310 | pkg/msg/wire_v2.go:143-150 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/msg/udp_binary.go"},"region":{"startLine":304}}}],"partialFingerprints":{"codehealthFindingId/v1":"ee31863febf8b0fabf4c417db584e998a69fc582634b7bcacbb23024cdf19a64"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (7 lines \u00D7 2): server/proxy/http.go:141-147 | server/proxy/udp.go:219-225 \u2014 the copies sit in sibling files of one directory: extract the block into a single shared function in that directory and call it from each site, so a change lands once. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/proxy/http.go"},"region":{"startLine":141}}}],"partialFingerprints":{"codehealthFindingId/v1":"71ff0d1aad0cd130fbbc1ab608721970675b6ea0ff60e4056c90ea70f0c28a3d"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): client/api_router.go:57-61 | server/api_router.go:63-68 \u2014 the copies span different directories, so extracting a shared function means choosing where it lives: put it somewhere both call sites can already reach \u2014 a location they all depend on today, or a new shared one if there is none \u2014 and call it from each site; until then, every change has to be made twice. Read the line range as the matched WINDOW rather than a finished unit: at \u0060client/api_router.go:57\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/api_router.go"},"region":{"startLine":57}}}],"partialFingerprints":{"codehealthFindingId/v1":"91dcb95dc789bb37ba7629a0d6d836b06b92c4944a1ddcd87e1f6ded9f29d0f6"}},{"ruleId":"D4","level":"warning","message":{"text":"Duplicated block (5 lines \u00D7 2): cmd/frpc/sub/admin.go:76-80 | cmd/frpc/sub/admin.go:116-120 \u2014 both copies are in the same file, so extract the block into one function there and call it from each site \u2014 the copies drift apart the first time only one of them is edited. Read the line range as the matched WINDOW rather than a finished unit: at \u0060cmd/frpc/sub/admin.go:76\u0060 it does not close everything it opens, so those exact lines cannot be lifted as they stand \u2014 widen the region to the smallest complete statement or declaration that contains it, and extract that. The matched lines also register a scope-exit action (a \u0060defer\u0060-style statement) that runs when the function holding them returns: moved into a called unit it would run when THAT unit returns instead \u2014 before the caller uses what it releases \u2014 so keep the registration at the call site and extract only the work around it, or have the extracted unit hand the resource back for the caller to register. Note that the copies do not run to the end of the range shown: their LAST lines are different code, not the same code under different names \u2014 the matched region ends inside that line. Extract the lines above it, and read the last line of each site separately."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"cmd/frpc/sub/admin.go"},"region":{"startLine":76}}}],"partialFingerprints":{"codehealthFindingId/v1":"933291ef40e5ddc6d1912ac5656a32b148fd9b6c99588ca373da0e13b61fc721"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: server/service.go: server/service.go changed 6 times in last 90 days, max complexity 30. 1 of those changes was a fix/bug commit, and the other 5 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, cover the area it touches with tests, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/service.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"b94d85e6e6326754253356c9dd5ef0e783fa621a06d6c7859cb1a00883f6b190"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: server/http/controller_v2.go: server/http/controller_v2.go changed 8 times in last 90 days, max complexity 17. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/http/controller_v2.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"a4c03f3b851a7fc4ac9724f4159f5bfa7d5124d987dbf6dbb13017cde51d0785"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: web/frps/src/views/Proxies.vue: web/frps/src/views/Proxies.vue changed 6 times in last 90 days, max complexity 15. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"web/frps/src/views/Proxies.vue"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"30fdcf2545ccfcfbcbdb8bb86c4e5f5082ceea110c7a36b0902063b9e16344eb"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: server/proxy/udp.go: server/proxy/udp.go changed 2 times in last 90 days, max complexity 27. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/proxy/udp.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"ab4f4e032dfc24d672e7fefe4dbc05689bc1a78a93a1005568ff244e49c62880"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: client/proxy/sudp.go: client/proxy/sudp.go changed 2 times in last 90 days, max complexity 17. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"client/proxy/sudp.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"05633b3293ccf58b3fddc21c32a0e26e8fa6bcaa9e9737edb48ecee11e028e20"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: server/registry/registry.go: server/registry/registry.go changed 2 times in last 90 days, max complexity 16. 1 of those changes was a fix/bug commit, and the other 1 changed it for other reasons \u2014 this file is under both repair and feature pressure. Before the next change lands here, cover the area it touches with tests, then split that area out of the file so the following change is smaller than this one \u2014 a file this often edited pays the complexity back every time."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"server/registry/registry.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"94c05b293ab65de8276b443647c86891e042eff33b1aea00d87ba55b3356292e"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: web/frps/src/views/ClientDetail.vue: web/frps/src/views/ClientDetail.vue changed 2 times in last 90 days, max complexity 15. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"web/frps/src/views/ClientDetail.vue"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"2e273ed0e6de6fceef94b21f24bfe6515e9a2a4cd336765f3377f32c73b1e566"}},{"ruleId":"D15","level":"warning","message":{"text":"Hotspot: web/frps/src/views/ProxyDetail.vue: web/frps/src/views/ProxyDetail.vue changed 2 times in last 90 days, max complexity 15. Frequent change and high complexity in one file compound: schedule the next change to it to include carving out the part being edited, behind tests written first."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"web/frps/src/views/ProxyDetail.vue"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"d0780d18bd74f273560ddce62311f1593248bb73bde8415b18839ed54c512810"}},{"ruleId":"D16","level":"note","message":{"text":"Off-boarding risk: anonymized user #1: If anonymized user #1 becomes unavailable, 49 significant file(s) lose their only recent owner: server/service.go, server/control.go, server/http/controller_v2.go, server/proxy/proxy.go, pkg/config/legacy/conversion.go, client/visitor/xtcp.go, pkg/nathole/controller.go, pkg/config/flags.go (\u002B41 more). Pair on, review, or document these before any departure."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"9ff6a28c707ac1499e2a73214cb123fc12a6aeb677bacc5cef3a566787f055e3"}},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"d2dc94fe31a395cfac39790d8360ac013b815b767b434d36c186534eeedfd837"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"fdeb0ea37134356d19b7bcd2c90e3d3990af774fd0094ce660a066ef31087575"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"8beab2c916b0443e4f77c354f18fbbc312ad6df83a4d06c562354ddd0f39ac79"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"19e4fbd34539f44dfed2bd94db1139ef819414c7b66697752c9debf5cb385ea3"},"taxa":[{"id":"CWE-78","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"d6bffe6cebd14aa8b2b95399acda5cdd0bb4c861cf25a8ea4b4981d5a904e93d"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"fd9b70e0fe56c1ac15882a38a73b953874e920b5f509873cfc3af7ef3ee081e3"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"c81f9a856ba60ba82e81d50acdc12bdb0577396a65e2d039248c815501d6546b"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"8b63077758bc28df6281889ac6762cbdc4dc0511526e8a810a631f85a8143aef"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"0a85b7f486877990f02b41cdbab4d4e0d982259e5fd68ac81e69a51984c290dc"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"8a8696033177e5f7c4729dba9961c91b2a1cf6e4ff0bdafbe8a2eb2916c847a5"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"d22714396275be3e78a62ce881e8a9ea7f5299d6a18f8a3af142432f2f0c3933"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"0a1b16554d7f044e5b64b829e299d230e6dfa4a22439a1a3fe3c953787e7f81d"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"d12ef85079892676ad44e782f6f032e8d0f54b88075a4d971b3c85a055a1d33b"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"a40d6baae94ac89d12cf27d1f36a504f70c043ea665341e2364333b22d2b223b"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"a878bc84e3dd41b04cf1773336168aac92acac9e39da8c962e76cc760c496612"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"1577a43e8096885d79e3856b1a9fb99d0274922316848e2b429fff717e11059b"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"ab8d83539c5ebe567cb6deae98d392d6617f9ea4001a34a9a21fb3f393c0be8f"},"taxa":[{"id":"CWE-1357","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}},{"id":"CWE-353","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"33c7aa2a4f90cc03c2dae338b1d8cac678a534c9422a94e1049d17b70b429108"},"taxa":[{"id":"CWE-328","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"be6580f4b3f4e4ad4abea09588251543f63120a2530975d12d5afd2a15701f4c"},"taxa":[{"id":"CWE-338","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"659de4e1b90c29c0ca80658faf52d67b01e669b5ae4a04a6d9d5d3388cf4ca01"},"taxa":[{"id":"CWE-328","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"1d623251f46531e86eab6bdc84fee6590cf616d1812890ea4bcff7f295c13fcb"},"taxa":[{"id":"CWE-338","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"6a1ed9c620668d6451805dca605587bd9513731a206eb09597d59a2c2a35215a"},"taxa":[{"id":"CWE-327","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"bd79baef4ff0a2f6ea041eddc452be1d2c575edf19bb9e1bad2f4ae0527c527f"},"taxa":[{"id":"CWE-327","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"47a15d8d82e288d4d2590b843744bc07c274888da69cfd74fb1e54d657bf50fa"},"taxa":[{"id":"CWE-327","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"b351a2ba111e9b34bb59be5963a435e18e47693164978d7fef94d0d016a1ef4f"},"taxa":[{"id":"CWE-327","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"b60b7074b5ed84d2552634c35b4ad48ace3f6c8121617ab9c1b0459c77d9321f"},"taxa":[{"id":"CWE-327","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"e0918da5a62a5b4caf32f502bb7418b6f0f09552271069ca28aab22db44bf7e5"},"taxa":[{"id":"CWE-327","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"db7993a9100db109992fe6e3700527770a5cf9a60b51026249f8e1ac22a886ab"},"taxa":[{"id":"CWE-327","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"dc6ab33cf75ec7f82e65059e2ddf7ffd1f1d1195f2e9a68a7ec2e7fb8ea9853e"},"taxa":[{"id":"CWE-327","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D29","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"45ea428f0c93040295d483edd2f5d997c0e9c32d38ff1ff91605093286f2af18"},"taxa":[{"id":"CWE-327","toolComponent":{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d"}}]},{"ruleId":"D34","level":"note","message":{"text":"Further orphaned files (smaller): 1 of 123 analysed file(s) have no living knowledge left \u2014 their last meaningful change has decayed away, so if one breaks, no one currently understands it (counted over production source files of roughly 100 lines or more, excluding tests, vendored, generated and example/demo trees, largest first). None is large enough to earn a read-through of its own, so this row stands in for the per-file rows rather than raising one each \u2014 largest first: pkg/util/http/http.go. Attach the read to the next change that touches one of them: have a second person review that change, and leave behind a short comment or test recording what the file is for, so the knowledge comes back at the cost of a change you were making anyway."},"locations":[],"partialFingerprints":{"codehealthFindingId/v1":"ce861fd78f6935114d3a342cb90ad25870f984e1042954fcbbe27c0e23be3658"}},{"ruleId":"D35","level":"warning","message":{"text":"Change coupling: client.go \u2194 server.go: \u0060pkg/config/legacy/client.go\u0060 and \u0060pkg/config/legacy/server.go\u0060 change together 71% of the time (12 of the 17 commits that touched whichever of the two files changed less often, counting a file under its earlier names as well). They sit in the same directory, and in this ecosystem sibling files there normally share one namespace/package \u2014 so a direct reference between them needs no import and this pass cannot see whether one exists. Read the pair before acting: if one file only DECLARES what the other consumes (a constants/types file beside its user), the co-change is definitional and the question is whether the split earns its keep; if they duplicate structure, extract the common part into a shared function or type they both call; if neither holds, the coupling is hidden and worth breaking."},"locations":[{"physicalLocation":{"artifactLocation":{"uri":"pkg/config/legacy/client.go"},"region":{"startLine":1}}}],"partialFingerprints":{"codehealthFindingId/v1":"887b749cd113b1ec6a7b02c0f8035295dbc726a78ad00aa49364d41e6e959429"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"1b213f6eedd4b140d0bc37bdf1496f72811a643f34064f12518b32e9e83bcfc7"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"0e17f71490e4d120a48b2b2881ab866c93b272bef2febb673a3e8e42b2c288ab"}},{"ruleId":"D36","level":"note","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"eb00976a698a5d68999b7ee6d27206fd374e916853e7ff1ead5eed42386f043f"}},{"ruleId":"D36","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"90f83b4fa27db32740afe9540c905f3c0499caed87f61049a8a903ecc95b41c3"}},{"ruleId":"D38","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"196e767899709fd49cb9b76f1686e6b44bd30d05446047c4892dec76125d87f9"}},{"ruleId":"D38","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"355bedf9e40482ac137be454010e5c5e1b90059c53b8ebecea862385d9666a1f"}},{"ruleId":"D38","level":"error","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"de29dcd1988e28b3d16ad39acef38672e2f985e1705d5603e9a2c5570096bde6"}},{"ruleId":"D38","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"e034055f3f1c4b2ccd9238992459db7e1d17f3108d878a4469e3a90784c80c77"}},{"ruleId":"D38","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"50c379534161bb30611d421e82af3666b891a67b9a6a35df5a4fc4b4977e653b"}},{"ruleId":"D38","level":"warning","message":{"text":"A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."},"partialFingerprints":{"codehealthFindingId/v1":"626e01eacf94c3b3382809ff8a91277880813d4497b12c2cdcf28b0d8c94458b"}}],"taxonomies":[{"name":"CWE","guid":"c3a2b1d0-7f3e-4b2a-9c1d-5e6f7a8b9c0d","organization":"MITRE","informationUri":"https://cwe.mitre.org/","isComprehensive":false,"shortDescription":{"text":"The MITRE Common Weakness Enumeration (CWE)."},"taxa":[{"id":"CWE-1357","guid":"e4d2e772-757e-0a5c-bd7d-77052949d866","name":"Reliance on Insufficiently Trustworthy Component","shortDescription":{"text":"Reliance on Insufficiently Trustworthy Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1357.html"},{"id":"CWE-1395","guid":"800e09e7-c11a-8654-9fa6-86f398995fed","name":"Dependency on Vulnerable Third-Party Component","shortDescription":{"text":"Dependency on Vulnerable Third-Party Component"},"helpUri":"https://cwe.mitre.org/data/definitions/1395.html"},{"id":"CWE-259","guid":"ae9ad959-fbb6-9d5e-892d-3dca66da0b69","name":"Use of Hard-coded Password","shortDescription":{"text":"Use of Hard-coded Password"},"helpUri":"https://cwe.mitre.org/data/definitions/259.html"},{"id":"CWE-327","guid":"e51910ff-67b5-3c51-aa3d-92837d9eab90","name":"CWE-327","shortDescription":{"text":"CWE-327"},"helpUri":"https://cwe.mitre.org/data/definitions/327.html"},{"id":"CWE-328","guid":"f0ffc869-97c3-dc5c-8825-be1159f1fc5f","name":"CWE-328","shortDescription":{"text":"CWE-328"},"helpUri":"https://cwe.mitre.org/data/definitions/328.html"},{"id":"CWE-338","guid":"e89b7604-c3f7-8b53-bc8c-4d4718c28617","name":"CWE-338","shortDescription":{"text":"CWE-338"},"helpUri":"https://cwe.mitre.org/data/definitions/338.html"},{"id":"CWE-353","guid":"09d7e902-d4ee-f05d-ae6c-0a1554d0c18f","name":"CWE-353","shortDescription":{"text":"CWE-353"},"helpUri":"https://cwe.mitre.org/data/definitions/353.html"},{"id":"CWE-494","guid":"b8a65e0d-e459-4a55-a931-fc1136482375","name":"Download of Code Without Integrity Check","shortDescription":{"text":"Download of Code Without Integrity Check"},"helpUri":"https://cwe.mitre.org/data/definitions/494.html"},{"id":"CWE-77","guid":"332c8ade-6612-9f56-a06b-d8d90b1a8750","name":"Command Injection","shortDescription":{"text":"Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/77.html"},{"id":"CWE-78","guid":"2e31ceaf-c7ae-2e5e-9661-cfb1362789cf","name":"OS Command Injection","shortDescription":{"text":"OS Command Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/78.html"},{"id":"CWE-79","guid":"fd45580b-e8c4-fc5e-8c2f-aa8fab0b4dbf","name":"Cross-site Scripting (XSS)","shortDescription":{"text":"Cross-site Scripting (XSS)"},"helpUri":"https://cwe.mitre.org/data/definitions/79.html"},{"id":"CWE-798","guid":"5e8f057d-fee3-995a-a0cb-9fc5b0d174d1","name":"Use of Hard-coded Credentials","shortDescription":{"text":"Use of Hard-coded Credentials"},"helpUri":"https://cwe.mitre.org/data/definitions/798.html"},{"id":"CWE-89","guid":"6d08fdad-37eb-c150-bbf0-d7d946863407","name":"SQL Injection","shortDescription":{"text":"SQL Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/89.html"},{"id":"CWE-937","guid":"16f316ae-415c-b354-a59b-1f7905f756e9","name":"Using Components with Known Vulnerabilities","shortDescription":{"text":"Using Components with Known Vulnerabilities"},"helpUri":"https://cwe.mitre.org/data/definitions/937.html"},{"id":"CWE-94","guid":"75e7f50c-6c2f-dd52-bf40-bf6c52b861fd","name":"Code Injection","shortDescription":{"text":"Code Injection"},"helpUri":"https://cwe.mitre.org/data/definitions/94.html"}]}],"properties":{"codehealthPublication":{"public":true,"notice":"This is the PUBLIC form of this artifact. Findings are listed in full, but the details of SECURITY findings \u2014 which rule fired, in which file, on which line, and how to fix it \u2014 are deliberately withheld, and any secret-scanner results are excluded entirely. Where detail is absent here it was REMOVED FOR PUBLICATION; it is not missing from the analysis. The complete artifact is available from the repository owner.","securityFindingsRedacted":40,"secretScannerRunsExcluded":0}},"redactionTokens":["A security finding was recorded here. Its details are withheld on the public artifact \u2014 ask the repository owner for the full report."]}]}