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

[BUG] DynamoDB Source doesn't support parsing data with Control Characters #5027

Open
dinujoh opened this issue Oct 7, 2024 · 0 comments · May be fixed by #5177
Open

[BUG] DynamoDB Source doesn't support parsing data with Control Characters #5027

dinujoh opened this issue Oct 7, 2024 · 0 comments · May be fixed by #5177
Labels
bug Something isn't working Priority-High

Comments

@dinujoh
Copy link
Member

dinujoh commented Oct 7, 2024

Describe the bug
The DynamoDB Source doesn't support parsing data with Control Characters

Object deserialization/serialization doesn't handle Control Characters:

https://github.com/opensearch-project/data-prepper/blob/main/data-prepper-plugins/dynamodb-source/src/main/java/org/opensearch/dataprepper/plugins/source/dynamodb/converter/StreamRecordConverter.java#L93

[pool-18-thread-104] ERROR org.opensearch.dataprepper.plugins.source.dynamodb.converter.StreamRecordConverter - Failed to parse and convert data from stream due to Illegal unquoted character ((CTRL-CHAR, code 4)): has to be escaped using backslash to be included in string value
 at [Source: REDACTED (`StreamReadFeature.INCLUDE_SOURCE_IN_LOCATION` disabled); line: 1, column: 720]

Expected behavior
DynamoDb source should escape the control characters and process the data.

@dinujoh dinujoh added bug Something isn't working untriaged labels Oct 7, 2024
@oeyh oeyh removed the untriaged label Oct 8, 2024
@paulsasi paulsasi linked a pull request Nov 8, 2024 that will close this issue
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working Priority-High
Projects
Development

Successfully merging a pull request may close this issue.

3 participants