Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Enhance CSV export with safer data handling#222

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
aouinizied merged 1 commit intonfstream:masterfromdrnpkr:csv-export-fix
Nov 25, 2025

Conversation

@drnpkr
Copy link
Member

This Enhancement

  • Replaces manual string concatenation withcsv.writer() forRFC 4180 compliance
  • Updatesopen_file() to use text mode with proper encoding and newline handling
  • Removes manual quoting fromcsv_converter() ascsv.writer() handles this properly

This addressespotential data corruption with special characters (commas, quotes, newlines) in network flow data by using Python's standard CSV module instead of manual string formatting.

Dependency

This enhancement requires importing thecsv module, which might be not desired. However, thecsv module is part of Python's standard library and has been included since Python 2.3. It is available in all Python versions without any pip installation required.

- Replace manual string concatenation with csv.writer() for RFC 4180 compliance- Update open_file() to use text mode with proper encoding and newline handling- Remove manual quoting from csv_converter() as csv.writer() handles this properlyThis addresses potential data corruption with special characters (commas, quotes,newlines) in network flow data by using Python's standard CSV module insteadof manual string formatting.
@aouiniziedaouinizied merged commit7939a98 intonfstream:masterNov 25, 2025
16 of 23 checks passed
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

@aouiniziedaouiniziedaouinizied approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@drnpkr@aouinizied

[8]ページ先頭

©2009-2025 Movatter.jp