Skip to content

fix: replace @JsonProperty with @JsonIgnore on request getter methods - #23

Closed
fern-api[bot] wants to merge 1 commit into
mainfrom
fern-bot/2026-04-13T21-54Z
Closed

fern-api[bot] wants to merge 1 commit into
mainfrom
fern-bot/2026-04-13T21-54Z

Conversation

@fern-api

@fern-api fern-api Bot commented Apr 13, 2026

Copy link
Copy Markdown
Contributor

Version Bump: These are internal serialization annotation corrections and a concurrency safety fix with no changes to public method signatures or observable API behavior.

Request classes across multiple endpoints were incorrectly annotating
their getter methods with @JsonProperty, which caused Jackson to attempt
to serialize these request objects as JSON bodies. Since these fields are
sent as query parameters (not request body fields), they should be
excluded from JSON serialization. Replacing @JsonProperty with @JsonIgnore
on all getters corrects this behavior across GetAggregationCountRequest,
GetAuthorsRequest, and GetBreakingNewsRequest.

Additionally, RefactoryInterceptor now creates a fresh ExponentialBackoff
instance per retry chain invocation instead of sharing a single instance,
preventing potential state leakage between concurrent requests.

Key changes:

  • Replace @JsonProperty with @JsonIgnore on all getter methods in GetAggregationCountRequest, GetAuthorsRequest, and GetBreakingNewsRequest
  • Create a new ExponentialBackoff instance per retryChain() call in RetryInterceptor instead of reusing a shared instance

🌿 Generated with Fern

Request classes across multiple endpoints were incorrectly annotating
their getter methods with @JsonProperty, which caused Jackson to attempt
to serialize these request objects as JSON bodies. Since these fields are
sent as query parameters (not request body fields), they should be
excluded from JSON serialization. Replacing @JsonProperty with @JsonIgnore
on all getters corrects this behavior across GetAggregationCountRequest,
GetAuthorsRequest, and GetBreakingNewsRequest.

Additionally, RefactoryInterceptor now creates a fresh ExponentialBackoff
instance per retry chain invocation instead of sharing a single instance,
preventing potential state leakage between concurrent requests.

Key changes:
- Replace @JsonProperty with @JsonIgnore on all getter methods in GetAggregationCountRequest, GetAuthorsRequest, and GetBreakingNewsRequest
- Create a new ExponentialBackoff instance per retryChain() call in RetryInterceptor instead of reusing a shared instance

🌿 Generated with Fern
@oscatcher oscatcher closed this Apr 23, 2026
@oscatcher
oscatcher deleted the fern-bot/2026-04-13T21-54Z branch May 1, 2026 16:07
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.

1 participant