Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

One-off API Production Deploy to fix CSV parsing issue - 02-02-2024 #786

Merged
merged 4 commits into from
Feb 2, 2024

Conversation

ccostino
Copy link
Contributor

@ccostino ccostino commented Feb 2, 2024

This is a one-off production deployment for the API to fix the parsing of CSV header rows for CSVs that are encoded with the utf8bom character set (UTF-8 with byte order marks). It also includes a fix for a log statement to be a proper f'' string (formatted string).

Security Considerations

  • There are no changes to the actual CSV processing for the batch notifications, just how the header row is read.
  • The log statement fix ensures our application log entries are formatted properly.

@ccostino ccostino requested a review from stvnrlly February 2, 2024 15:20
@ccostino ccostino self-assigned this Feb 2, 2024
@stvnrlly stvnrlly merged commit 1ca6070 into production Feb 2, 2024
10 of 13 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants