Skip to content

Commit 812b5cd

Browse files
Merge pull request #284 from Yamato-Security/282-fix-commna
fix: Correct CSV header output handling
2 parents 48a8b38 + deca719 commit 812b5cd

File tree

2 files changed

+4
-6
lines changed

2 files changed

+4
-6
lines changed

src/takajopkg/timelineLogon.nim

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -400,9 +400,8 @@ method resultOutput*(self: TimelineLogonCmd) =
400400
"TargetUserSID", "TargetDomainName", "TargetLinkedLID"]
401401

402402
## Write CSV header
403-
for h in header:
404-
outputFile.write(h & ",")
405-
outputFile.write("\p")
403+
let headerLine = header.join(",")
404+
outputFile.write(headerLine & "\p")
406405

407406
## Write contents
408407
self.seqOfResultsTables.sort(proc(a, b: TableRef[string, string]): int =

src/takajopkg/timelineTasks.nim

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -78,9 +78,8 @@ method resultOutput*(self: TimelineTasksCmd) =
7878

7979
## Write CSV header
8080
let header = concat(basicHeader, detailedHeader)
81-
for h in header:
82-
outputFile.write(h & ",")
83-
outputFile.write("\p")
81+
let headerLine = header.join(",")
82+
outputFile.write(headerLine & "\p")
8483

8584
## Write contents
8685
for (basicTable, detailedTable) in self.rowData:

0 commit comments

Comments
 (0)