mirror of
https://github.com/norandom/OpenUltraSAST.git
synced 2026-10-19 21:19:28 +00:00
Marius CiepluchandClaude Opus 5
c2d9c4ad01
Eight of the ten vendored access-control rows had no named handler at all, so every one of them would have scored `missing_context` forever rather than as a detector miss. After the re-harvest nine are named and the tenth states its reason. The handler-context mode now carries the class its framework registers, multi-line decorators, and commented-out guards -- `#@token_required` above a ThreatByte handler is the entire bug. Where the registration lives in another document (connexion's `operationId`, an express router), a recipe names `context = [...]`, the harvester writes that stanza per side, and the pair materializes both files at their repository-relative paths so the entry-point mapper sees the layout the project has. `harvest_recipe` fetches every blob before the first write and the driver takes `--report`, so a failed fetch leaves a reviewed excerpt exactly as it was. Two defects the re-harvest exposed: `.mjs` was not a known suffix, so the one pair vendored as an ES module had no file target at all; and `_python_route_access` matched auth words inside decorator prose, reading `@ns.doc(description='... without proper authorization checks')` as a guard. Recorded in the hygiene artifact: nine of the ten duplicate groups sit on both sides of the split, which the repository-level split cannot see. Co-Authored-By: Claude Opus 5 (1M context) <[email protected]> Claude-Session: https://claude.ai/code/session_01T7XD7x5KonMdmGRkp63EQG
72 lines
2.7 KiB
TypeScript
72 lines
2.7 KiB
TypeScript
// Provenance: amitknigam84/lensops loadNamespaces (fixed).
|
|
// repo: amitknigam84/lensops
|
|
// commit: fe49109aedfe21dea4d24d4d9f2fa3f1ecfb6c84
|
|
// parent: 9b87ecc3a44a7fe2a5185cca1fe52441647d5748
|
|
// commit_url: https://github.com/amitknigam84/lensops/commit/fe49109aedfe21dea4d24d4d9f2fa3f1ecfb6c84
|
|
// cve:
|
|
// license: MIT
|
|
// function: loadNamespaces
|
|
// relpath: components/KubernetesManagement.tsx
|
|
// provenance: agent
|
|
// mechanism: missing_auth_guard
|
|
// upstream_start: 805
|
|
|
|
const loadNamespaces = async (skipConnectionCheck = false) => {
|
|
if (!skipConnectionCheck && !isConnected) {
|
|
setErrorMessage('Please connect to a Kubernetes cluster first')
|
|
return
|
|
}
|
|
|
|
setIsLoading(true)
|
|
setErrorMessage(null)
|
|
|
|
try {
|
|
console.log('📋 Loading namespaces from:', `${apiUrl}/k8s/namespaces`)
|
|
const response = await fetch(`${apiUrl}/k8s/namespaces`, {
|
|
headers: token ? { 'Authorization': `Bearer ${token}` } : {},
|
|
})
|
|
|
|
if (!response.ok) {
|
|
const errorData = await response.json().catch(() => ({ message: 'Unknown error' }))
|
|
throw new Error(errorData.message || `HTTP ${response.status}: ${response.statusText}`)
|
|
}
|
|
|
|
const data = await response.json()
|
|
console.log('📋 Namespaces response:', data)
|
|
|
|
if (data.success && data.namespaces) {
|
|
setNamespaces(data.namespaces)
|
|
setIsConnected(true) // ✅ Mark as connected after successfully loading namespaces
|
|
|
|
// Always use deployed namespace (from ServiceAccount)
|
|
const namespaceToUse = data.podNamespace || (data.namespaces.length > 0 ? data.namespaces[0].name : null);
|
|
|
|
if (namespaceToUse) {
|
|
setDeployedNamespace(namespaceToUse)
|
|
setSelectedNamespace(namespaceToUse)
|
|
console.log('📌 Using deployed namespace:', namespaceToUse)
|
|
// Automatically load resources for the deployed namespace
|
|
console.log('📦 Loading resources from deployed namespace:', namespaceToUse)
|
|
setTimeout(() => {
|
|
loadData()
|
|
}, 100)
|
|
} else {
|
|
setErrorMessage('⚠️ No namespace found. Please check POD_NAMESPACE configuration.')
|
|
}
|
|
} else {
|
|
setErrorMessage(data.message || data.error || 'Failed to load namespaces')
|
|
}
|
|
} catch (error: any) {
|
|
console.error('❌ Failed to load namespaces:', error)
|
|
setErrorMessage(error.message || 'Failed to load namespaces. Please check your connection and try again.')
|
|
} finally {
|
|
setIsLoading(false)
|
|
}
|
|
}
|
|
|
|
await loadNamespaces(true)
|
|
await loadNamespaces(true)
|
|
loadNamespaces(true)
|
|
loadNamespaces(true) // Skip connection check since we're auto-connecting
|
|
loadNamespaces(true);
|