Skip to content

Commit

Permalink
fix yaml parsing
Browse files Browse the repository at this point in the history
  • Loading branch information
code-crusher committed Jan 29, 2025
1 parent 16219c7 commit 4475ae8
Showing 1 changed file with 18 additions and 24 deletions.
42 changes: 18 additions & 24 deletions src/index.ts
Original file line number Diff line number Diff line change
Expand Up @@ -699,29 +699,23 @@ const sendDetailsToAgentJob = async (details: any) => {
]

// Get all environment variables that use secretKeyRef, excluding known secrets
const secretEnvs = Object.entries(process.env)
.filter(([key, value]) => {
// Skip known secret env vars
if (knownSecretEnvs.includes(key)) return false

try {
const parsed = JSON.parse(value ?? '')
// Check if it's a secretKeyRef structure
return parsed?.valueFrom?.secretKeyRef?.name && parsed?.valueFrom?.secretKeyRef?.key
} catch {
return false
}
})
.map(([name, value]) => {
const parsed = JSON.parse(value ?? '')
return `
- name: ${name}
valueFrom:
secretKeyRef:
name: ${parsed.valueFrom.secretKeyRef.name}
key: ${parsed.valueFrom.secretKeyRef.key}`
})
.join('\n')
const secretEnvsYaml = Object.entries(process.env)
.filter(([key, value]) => {
if (knownSecretEnvs.includes(key)) return false
try {
const parsed = JSON.parse(value ?? '')
return parsed?.valueFrom?.secretKeyRef?.name && parsed?.valueFrom?.secretKeyRef?.key
} catch {
return false
}
})
.map(([name, value]) => {
const parsed = JSON.parse(value ?? '')
return {
name,
valueFrom: parsed.valueFrom
}
})

const jobTemplate = `apiVersion: batch/v1
kind: Job
Expand Down Expand Up @@ -751,7 +745,7 @@ spec:
readOnly: true
env:
${predefinedSecrets}
${secretEnvs}
${secretEnvsYaml}
- name: GRAVITY_WEBSOCKET_URL
value: "${process.env.GRAVITY_WEBSOCKET_URL}"
- name: GRAVITY_API_URL
Expand Down

0 comments on commit 4475ae8

Please sign in to comment.