search

contentful / contentful-sdk-core

Core modules for the Contentful JS SDKs
MIT License
22 stars 20 forks source link

fix: do not remove query parameters on retry #514

Closed spzSource closed 8 hours ago

spzSource commented 2 weeks ago

Fixes #513. This is a draft fix intended to start the discussion.

I'm not sure if it's okay to remove the existing hack introduced in #483. Before making a decision, I’d like to learn more about the duplicate request parameters problem and how I can reproduce it in a test.

@t-col probably you could give me some hints?

spzSource commented 9 hours ago

I’ve committed a fix for the formatting issue that caused the CI build to fail, so I believe everything is ready for the merge.

contentful-automation[bot] commented 7 hours ago

:tada: This PR is included in version 8.3.2 :tada:

The release is available on:

Your semantic-release bot :package::rocket: