Skip to content

Encode form name#211

Merged
labkey-nicka merged 4 commits intodevelopfrom
fb_encodeFormName
Mar 30, 2026
Merged

Encode form name#211
labkey-nicka merged 4 commits intodevelopfrom
fb_encodeFormName

Conversation

@labkey-nicka
Copy link
Copy Markdown
Contributor

@labkey-nicka labkey-nicka commented Mar 26, 2026

Rationale

See LabKey/platform#7515 for rationale.

Related Pull Requests

Changes

  • Introduce Utils.encodeFormName which is a client-side corollary for PageFlowUtil.encodeFormName.
  • Update form data binding to encode keys
  • Unit tests

@labkey-nicka labkey-nicka merged commit 67694fc into develop Mar 30, 2026
3 checks passed
@labkey-nicka labkey-nicka deleted the fb_encodeFormName branch March 30, 2026 22:06
labkey-matthewb added a commit to LabKey/platform that referenced this pull request Mar 30, 2026
#### Rationale
Whereas browsers seem to have trouble property formatting
multipart/form-data POSTs, we will encode form names that include these
three characters: `% " \`

For backward compatibility we will identify encoded field names by
prepending `%_`.

`quf_name` -> `quf_name`
`quf_why"?` -> `%_quf_why%22%3F`

#### Related Pull Requests
- LabKey/labkey-api-js#211
- LabKey/labkey-ui-components#1961
- LabKey/labkey-ui-premium#916
- LabKey/limsModules#2061
- #7515
- LabKey/testAutomation#2923

#### Changes
- <!-- list of descriptions of changes that are worth noting (replace
this comment) -->

---------

Co-authored-by: labkey-nicka <nickk@labkey.com>
labkey-matthewb added a commit to LabKey/testAutomation that referenced this pull request Mar 30, 2026
#### Rationale
Support an encoding scheme for INPUT.name to avoid problems with
multipart/form-data

#### Related Pull Requests
- LabKey/labkey-api-js#211
- LabKey/labkey-ui-components#1961
- LabKey/labkey-ui-premium#916
- LabKey/limsModules#2061
- LabKey/platform#7515
- #2923

#### Changes
- <!-- list of descriptions of changes that are worth noting (replace
this comment) -->

---------

Co-authored-by: labkey-nicka <nickk@labkey.com>
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