Skip to content

Comments

fix(proxy): [ExportRequestType] support camelCase and upper SNAKE_CASE#720

Closed
yasinmiran wants to merge 2 commits intomainfrom
fix/camel-and-upper-snake-case
Closed

fix(proxy): [ExportRequestType] support camelCase and upper SNAKE_CASE#720
yasinmiran wants to merge 2 commits intomainfrom
fix/camel-and-upper-snake-case

Conversation

@yasinmiran
Copy link
Contributor

The ExportRequestType enum only accepted camelCase values ("fileId", "datasetId") during deserialization, causing requests with upper snake case ("FILE_ID", "DATASET_ID") to fail with an IllegalArgumentException. Updated fromValue() to also match against enum constant names. Added Javadoc to the class and unit tests for the deserialization logic.

@yasinmiran yasinmiran force-pushed the fix/camel-and-upper-snake-case branch from 7f869c5 to 63503a8 Compare February 13, 2026 14:02
@yasinmiran yasinmiran changed the title fix(proxy): [ExportRequestType] support camel case and upper snake fix(proxy): [ExportRequestType] support camelCase and upper SNAKE_CASE Feb 18, 2026
The ExportRequestType enum only accepted camelCase values ("fileId", "datasetId")
during deserialization, causing requests with upper snake case ("FILE_ID", "DATASET_ID")
to fail with an IllegalArgumentException. Updated fromValue() to also match against
enum constant names. Added Javadoc to the class and unit tests for the deserialization logic.
@yasinmiran yasinmiran force-pushed the fix/camel-and-upper-snake-case branch from 63503a8 to f855c33 Compare February 18, 2026 08:28
@yasinmiran yasinmiran marked this pull request as draft February 19, 2026 09:20
@yasinmiran yasinmiran closed this Feb 19, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants