bergerb / bergerb/OneRosterSampleDataGenerator

CSV file encoding not explicitly set to UTF-8

Open
#63 0 comments 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
C#
Stars
4
Forks
1
PR merge metrics
No merged PRs in 30d

Description

## Description

\FileProcessor.cs\ uses \
ew StreamWriter(filePath)\ which defaults to UTF-8 without BOM. However, the [OneRoster 1.1 Best Practice Guide (section 4.1)](https://www.imsglobal.org/oneroster-v11-final-best-practice-and-implementation-guide) requires:

> 'The CSV files must be UTF-8 encoded [RFC3629].'

While the default StreamWriter behavior produces UTF-8, being explicit avoids ambiguity across platforms.

## File to fix

\FileProcessor.cs\ line 25 — change to:

\\\csharp
using var writer = new StreamWriter(filePath, false, System.Text.Encoding.UTF8);
\\\

## Impact

Low — current output is already UTF-8, but explicit encoding removes any ambiguity.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.