Revert "[functions] URL encode cache tags"#15213
Conversation
This reverts commit 81336d6.
📦 CLI Tarball ReadyThe Vercel CLI tarball for this PR is now available! Quick TestYou can test this PR's CLI directly by running: npx https://vercel-9xzxzc912.vercel.sh/tarballs/vercel.tgz --helpUse in vercel.jsonTo use this CLI version in your project builds, add to your {
"build": {
"env": {
"VERCEL_CLI_VERSION": "vercel@https://vercel-9xzxzc912.vercel.sh/tarballs/vercel.tgz"
}
}
}Python Runtime WheelA Python runtime wheel was also built for this PR. |
🧪 Unit Test StrategyComparing: Strategy: Affected packages only ✅ Only testing packages that have been modified or depend on modified packages. Affected packages - 11 (28%)
Unaffected packages - 29 (73%)
Results
This comment is automatically generated based on the affected testing strategy |
🦋 Changeset detectedLatest commit: 83d1d76 The changes in this PR will be included in the next version bump. This PR includes changesets to release 1 package
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 |
|
@QuiiBz here is the relevant thread for the original PR that this is reverting: https://vercel.slack.com/archives/C02K2HCH5V4/p1769213460081909 We discussed today and came to the conclusion that this was the wrong change and we should not support commas in tags. |
This PR was opened by the [Changesets release](https://github.com/changesets/action) GitHub action. When you're ready to do a release, you can merge this and the packages will be published to npm automatically. If you're not ready to do a release yet, that's fine, whenever you add more changesets to main, this PR will be updated. # Releases ## vercel@50.23.0 ### Minor Changes - [dev] allow to skip authentication and project linking for `vc dev` by setting `VERCEL_EXPERIMENTAL_DEV_SKIP_LINK` env var. ([#15122](#15122)) - [services] detect project root to prevent accidental setup of a service as a standalone project. ([#15187](#15187)) ### Patch Changes - Fix re-authentication for teams with missing scope to use the device code flow instead of the deprecated SSO redirect flow. ([#15074](#15074)) - Add CLI eval for vc env command ([#15118](#15118)) - Updated dependencies \[]: - @vercel/node@5.6.7 ## @vercel/functions@3.4.3 ### Patch Changes - [functions] Revert "[functions] URL encode cache tags" ([#15213](#15213)) <!-- VADE_RISK_START --> > [!NOTE] > Low Risk Change > > This PR contains only version bumps and changelog updates from the Changesets release automation, with no actual code changes. > > - Deletes changeset markdown files consumed during release > - Updates version numbers in package.json files (50.22.3→50.23.0, 3.4.2→3.4.3) > - Adds changelog entries documenting previously merged changes > > <sup>Risk assessment for [commit 18f0132](https://github.com/vercel/vercel/commit/18f013287efb10cb34e666236f9e34f0c72d0c83).</sup> <!-- VADE_RISK_END --> Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
Reverts #14749 since this caused a regression when using
:and other characters in tags.The expected behavior is that commas split into multiple tags.
Warning
High Risk Change
Revert removes URL encoding of cache tags, which could allow special characters like commas to be interpreted as tag delimiters, potentially affecting cache invalidation behavior.
Risk assessment for commit 83d1d76.