update

Signed-off-by: laurentsimon <laurentsimon@google.com>

.github/workflows/generator_generic_slsa3.yml

@@ -332,10 +332,4 @@ jobs:
           [ "$CONTINUE" == "true" ] || [ "$SUCCESS" == "true" ] || exit 27
 
   # cleanup deletes internal artifacts used by the generator workflow
-  cleanup:
-    runs-on: ubuntu-latest
-    needs: [detect-env, generator, upload-assets]
-    steps:
-      - uses: geekyeggo/delete-artifact@54ab544f12cdb7b71613a16a2b5a37a9ade990af # v2.0.0
-        with:
-          name: "${{ needs.generator.outputs.subject-artifact-name }}"
+  # TODO(#2382): Delete artifacts ${{ needs.generator.outputs.subject-artifact-name }}

internal/builders/generic/attest.go

@@ -39,7 +39,6 @@ func attestCmd(provider slsa.ClientProvider, check func(error),
 ) *cobra.Command {
 	var attPath string
 	var subjectsFilename string
-	var subjects string
 
 	c := &cobra.Command{
 		Use:   "attest",
@@ -52,13 +51,9 @@ run in the context of a Github Actions workflow.`,
 			ghContext, err := github.GetWorkflowContext()
 			check(err)
 
-			// If not sujects are provided, read from the file instead.
-			if subjects == "" {
-				subjectsBytes, err := utils.SafeReadFile(subjectsFilename)
-				check(err)
-				subjects = string(subjectsBytes)
-			}
-			parsedSubjects, err := parseSubjects(subjects)
+			subjectsBytes, err := utils.SafeReadFile(subjectsFilename)
+			check(err)
+			parsedSubjects, err := parseSubjects(string(subjectsBytes))
 			check(err)
 			if len(parsedSubjects) == 0 {
 				check(errors.New("expected at least one subject"))
@@ -142,9 +137,5 @@ run in the context of a Github Actions workflow.`,
 		&subjectsFilename, "subjects-filename", "f", "",
 		"Filename containing a formatted list of subjects in the same format as sha256sum (base64 encoded).",
 	)
-	c.Flags().StringVarP(
-		&subjects, "subjects", "s", "",
-		"A formatted list of subjects in the same format as sha256sum (base64 encoded).",
-	)
 	return c
 }