CSV file encoding not explicitly set to UTF-8
Nobody has claimed this yet.
Assessment
- Difficulty
- 1/5
- Estimated time
- Under an hour
- Newbie friendliness
- 88/100
Research direction
Open FileProcessor.cs at line 25 and inspect how the StreamWriter is constructed. Make the encoding explicit as UTF-8, then verify that generated CSV output remains UTF-8 encoded and that the existing project checks still pass.
Written by the indexing model from the issue text.
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) 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.
- Dominant language
- C#
- Stars
- 4
- Forks
- 1
- PR merge metrics
- No merged PRs in 30d
Contributor guide
No contributing guide indexed for this repository
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from bergerb/OneRosterSampleDataGenerator
-
enhancement
Difficulty 1/5 Under an hour Newbie friendliness 90/100
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
-
bug
Difficulty 1/5 Under an hour Newbie friendliness 78/100
-
bug
Difficulty 4/5 3-5 days Newbie friendliness 48/100
-
enhancement
Difficulty 5/5 Over a week Newbie friendliness 45/100
All issues in bergerb/OneRosterSampleDataGenerator
Similar issues
-
type/automation type/tech-debt
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
-
bug
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
-
t/bug
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
ci-failure-cause test-failure
Difficulty 2/5 1-3 hours Newbie friendliness 82/100
-
area:auth FE mvp P3
Difficulty 2/5 1-3 hours Newbie friendliness 88/100
klasolsson81/jobbliggaren#1788 ·