code changes to use yaml.loadAll and upgrade of octokit version

package.json

@@ -16,8 +16,8 @@
       "@actions/exec": "^1.0.0",
       "@actions/io": "^1.1.3",
       "@actions/tool-cache": "2.0.1",
-      "@octokit/core": "^6.1.2",
-      "@octokit/plugin-retry": "^7.1.2",
+      "@octokit/core": "^3.5.1",
+      "@octokit/plugin-retry": "^3.0.9",
       "@types/minipass": "^3.3.5",
       "js-yaml": "4.1.0",
       "minimist": "^1.2.8"

src/strategyHelpers/blueGreen/blueGreenHelper.ts

@@ -78,7 +78,7 @@ export function getManifestObjects(filePaths: string[]): BlueGreenManifests {
    // organized before we can check if services are “routed” or not.
    filePaths.forEach((filePath: string) => {
       const fileContents = fs.readFileSync(filePath).toString()
-      yaml.safeLoadAll(fileContents, (inputObject) => {
+      yaml.loadAll(fileContents, (inputObject: any) => {
          if (!!inputObject) {
             const kind = inputObject.kind
 

src/strategyHelpers/canary/canaryHelper.ts

@@ -213,7 +213,7 @@ async function cleanUpCanary(
    for (const filePath of files) {
       const fileContents = fs.readFileSync(filePath).toString()
 
-      const parsedYaml = yaml.safeLoadAll(fileContents)
+      const parsedYaml: any[] = yaml.loadAll(fileContents)
       for (const inputObject of parsedYaml) {
          const name = inputObject.metadata.name
          const kind = inputObject.kind

src/strategyHelpers/canary/podCanaryHelper.ts

@@ -22,7 +22,7 @@ export async function deployPodCanary(
 
    for (const filePath of filePaths) {
       const fileContents = fs.readFileSync(filePath).toString()
-      const parsedYaml = yaml.safeLoadAll(fileContents)
+      const parsedYaml = yaml.loadAll(fileContents)
       for (const inputObject of parsedYaml) {
          const name = inputObject.metadata.name
          const kind = inputObject.kind

src/strategyHelpers/canary/smiCanaryHelper.ts

@@ -11,6 +11,7 @@ import {isDeploymentEntity, isServiceEntity} from '../../types/kubernetesTypes'
 import {checkForErrors} from '../../utilities/kubectlUtils'
 import {inputAnnotations} from '../../inputUtils'
 import {DeployResult} from '../../types/deployResult'
+import { K8sObject } from '../../types/k8sObject'
 
 const TRAFFIC_SPLIT_OBJECT_NAME_SUFFIX = '-workflow-rollout'
 const TRAFFIC_SPLIT_OBJECT = 'TrafficSplit'
@@ -37,7 +38,7 @@ export async function deploySMICanary(
    const newObjectsList = []
    for await (const filePath of filePaths) {
       const fileContents = fs.readFileSync(filePath).toString()
-      const inputObjects = yaml.safeLoadAll(fileContents)
+      const inputObjects: K8sObject[] = yaml.loadAll(fileContents) as K8sObject[]
       for (const inputObject of inputObjects) {
          const name = inputObject.metadata.name
          const kind = inputObject.kind
@@ -112,7 +113,7 @@ async function createCanaryService(
 
    for (const filePath of filePaths) {
       const fileContents = fs.readFileSync(filePath).toString()
-      const parsedYaml = yaml.safeLoadAll(fileContents)
+      const parsedYaml = yaml.loadAll(fileContents)
       for (const inputObject of parsedYaml) {
          const name = inputObject.metadata.name
          const kind = inputObject.kind
@@ -225,7 +226,7 @@ async function adjustTraffic(
    const trafficSplitManifests = []
    for (const filePath of manifestFilePaths) {
       const fileContents = fs.readFileSync(filePath).toString()
-      const parsedYaml = yaml.safeLoadAll(fileContents)
+      const parsedYaml = yaml.loadAll(fileContents)
       for (const inputObject of parsedYaml) {
          const name = inputObject.metadata.name
          const kind = inputObject.kind

src/strategyHelpers/deploymentHelper.ts

@@ -112,7 +112,7 @@ function appendStableVersionLabelToResource(files: string[]): string[] {
    files.forEach((filePath: string) => {
       const fileContents = fs.readFileSync(filePath).toString()
 
-      yaml.safeLoadAll(fileContents, function (inputObject) {
+      yaml.loadAll(fileContents, function (inputObject) {
          const {kind} = inputObject
 
          if (isDeploymentEntity(kind)) {
@@ -197,7 +197,7 @@ async function annotateResources(
       for (const filePath of files) {
          core.debug('printing objects getting annotated...')
          const fileContents = fs.readFileSync(filePath).toString()
-         const inputObjects = yaml.safeLoadAll(fileContents)
+         const inputObjects = yaml.loadAll(fileContents)
          for (const inputObject of inputObjects) {
             core.debug(`object: ${JSON.stringify(inputObject)}`)
          }

src/utilities/fileUtils.test.ts

@@ -3,14 +3,16 @@ import * as fileUtils from './fileUtils'
 import * as yaml from 'js-yaml'
 import * as fs from 'fs'
 import * as path from 'path'
+import { K8sObject } from '../types/k8sObject';
 
 const sampleYamlUrl =
    'https://raw.githubusercontent.com/kubernetes/website/main/content/en/examples/controllers/nginx-deployment.yaml'
 describe('File utils', () => {
    test('correctly parses a yaml file from a URL', async () => {
       const tempFile = await fileUtils.writeYamlFromURLToFile(sampleYamlUrl, 0)
       const fileContents = fs.readFileSync(tempFile).toString()
-      const inputObjects = yaml.safeLoadAll(fileContents)
+      const inputObjects: K8sObject[] = yaml.loadAll(fileContents) as K8sObject[]; // Type assertion here
+
       expect(inputObjects).toHaveLength(1)
 
       for (const obj of inputObjects) {

src/utilities/fileUtils.ts

@@ -183,7 +183,7 @@ function verifyYaml(filepath: string, url: string): Errorable<K8sObject[]> {
    const fileContents = fs.readFileSync(filepath).toString()
    let inputObjects
    try {
-      inputObjects = yaml.safeLoadAll(fileContents)
+      inputObjects = yaml.loadAll(fileContents)
    } catch (e) {
       return {
          succeeded: false,

src/utilities/manifestUpdateUtils.ts

@@ -20,6 +20,7 @@ import {
    setImagePullSecrets
 } from './manifestPullSecretUtils'
 import {Resource} from '../types/kubectl'
+import { K8sObject } from '../types/k8sObject';
 
 export function updateManifestFiles(manifestFilePaths: string[]) {
    if (manifestFilePaths?.length === 0) {
@@ -275,7 +276,8 @@ export function getResources(
    const resources: Resource[] = []
    filePaths.forEach((filePath: string) => {
       const fileContents = fs.readFileSync(filePath).toString()
-      yaml.safeLoadAll(fileContents, (inputObject) => {
+      const inputObjects: K8sObject[] = yaml.loadAll(fileContents) as K8sObject[]
+      inputObjects.forEach((inputObject) => {
          const inputObjectKind = inputObject?.kind || ''
          if (
             filterResourceTypes.filter(
@@ -303,7 +305,7 @@ function updateImagePullSecretsInManifestFiles(
    const newObjectsList = []
    filePaths.forEach((filePath: string) => {
       const fileContents = fs.readFileSync(filePath).toString()
-      yaml.safeLoadAll(fileContents, (inputObject: any) => {
+      yaml.loadAll(fileContents, (inputObject: any) => {
          if (inputObject?.kind) {
             const {kind} = inputObject
             if (isWorkloadEntity(kind)) {