Skip to content

fix(snippetz): keep every value of a repeated form field in php/guzzle examples - #10499

Merged
hanspagel merged 2 commits into
scalar:mainfrom
charan-rathore:fix/snippetz-guzzle-repeated-form-fields
Oct 6, 2026
Merged

hanspagel merged 2 commits into
scalar:mainfrom
charan-rathore:fix/snippetz-guzzle-repeated-form-fields

Conversation

@charan-rathore

@charan-rathore charan-rathore commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

A URL-encoded form with repeated names loses data in the PHP Guzzle example. For tag=a&tag=b, the example previously sent only the last value. Guzzle also encodes arrays in form_params as indexed names, which would send tag[0]=a&tag[1]=b instead of the original fields.

Generate an encoded raw body when names repeat, preserving field names, order, empty values and URL encoding. Add the form Content-Type when absent and preserve an existing header. Forms without repeated names continue to use form_params.

@changeset-bot

changeset-bot Bot commented Oct 5, 2026 •

Copy link
Copy Markdown

🦋 Changeset detected

Latest commit: 62f923a

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 25 packages
Name Type
@scalar/snippetz Patch
@scalar/java-integration Patch
@scalar/api-client Patch
@scalar/api-reference Patch
@scalar/blocks Patch
@scalar/workspace-store Patch
@scalar/nuxt Patch
@scalar/agent-chat Patch
@scalar/api-client-react Patch
scalar-app Patch
@scalarapi/docker-api-reference Patch
@scalar/aspire Patch
@scalar/aspnetcore Patch
@scalar/aws-lambda Patch
@scalar/azure-functions Patch
@scalar/fastify-api-reference Patch
scalar_api_reference Patch
@scalar/api-reference-react Patch
@scalar/server-side-rendering Patch
@scalar/mock-server Patch
@scalar/oas-utils Patch
@scalar/openapi-to-markdown Patch
@scalar/pre-post-request-scripts Patch
@scalar/sidebar Patch
@scalar/mock-server-docker Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@hanspagel
hanspagel force-pushed the fix/snippetz-guzzle-repeated-form-fields branch from 813f4f0 to d634fa4 Compare October 6, 2026 09:31
@hanspagel
hanspagel temporarily deployed to production-netlify October 6, 2026 09:34 — with GitHub Actions Inactive
@hanspagel
hanspagel temporarily deployed to production-netlify October 6, 2026 09:34 — with GitHub Actions Inactive
@hanspagel
hanspagel temporarily deployed to production-netlify October 6, 2026 09:34 — with GitHub Actions Inactive
@hanspagel
hanspagel temporarily deployed to production-netlify October 6, 2026 09:34 — with GitHub Actions Inactive
@hanspagel
hanspagel force-pushed the fix/snippetz-guzzle-repeated-form-fields branch from d634fa4 to 62f923a Compare October 6, 2026 09:40
@hanspagel
hanspagel temporarily deployed to production-netlify October 6, 2026 09:41 — with GitHub Actions Inactive
@hanspagel
hanspagel temporarily deployed to production-netlify October 6, 2026 09:41 — with GitHub Actions Inactive
@hanspagel
hanspagel temporarily deployed to production-netlify October 6, 2026 09:41 — with GitHub Actions Inactive
@hanspagel
hanspagel temporarily deployed to production-netlify October 6, 2026 09:41 — with GitHub Actions Inactive
@hanspagel
hanspagel temporarily deployed to production-netlify October 6, 2026 09:41 — with GitHub Actions Inactive
@hanspagel
hanspagel enabled auto-merge (squash) October 6, 2026 09:41
@hanspagel
hanspagel merged commit 5356459 into scalar:main Oct 6, 2026
31 checks passed
@charan-rathore
charan-rathore deleted the fix/snippetz-guzzle-repeated-form-fields branch October 6, 2026 09:46
@scalar-release-bot scalar-release-bot Bot mentioned this pull request Oct 6, 2026

This branch was previously deployed

1 inactive deployment
production-netlify — 62f923a0 Deployed Oct 6, 2026 by hanspagel via test-block-snapshots #3210
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