-
Notifications
You must be signed in to change notification settings - Fork 194
fix: added encoding on absolutePointer before decodeURIComponent #2461
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
🦋 Changeset detectedLatest commit: 666c5e0 The changes in this PR will be included in the next version bump. This PR includes changesets to release 3 packages
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 |
|
Coverage Report
File Coverage
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||
|
📦 A new experimental 🧪 version v0.0.0-snapshot.1765541222 of Redocly CLI has been published for testing. Install with NPM: npm install @redocly/[email protected]
# or
npm install @redocly/[email protected]
# or
npm install @redocly/[email protected] |
|
📦 A new experimental 🧪 version v0.0.0-snapshot.1765561541 of Redocly CLI has been published for testing. Install with NPM: npm install @redocly/[email protected]
# or
npm install @redocly/[email protected]
# or
npm install @redocly/[email protected] |
kanoru3101
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
54cbc2e to
0452ee1
Compare
|
📦 A new experimental 🧪 version v0.0.0-snapshot.1765888495 of Redocly CLI has been published for testing. Install with NPM: npm install @redocly/[email protected]
# or
npm install @redocly/[email protected]
# or
npm install @redocly/[email protected] |
|
📦 A new experimental 🧪 version v0.0.0-snapshot.1765905976 of Redocly CLI has been published for testing. Install with NPM: npm install @redocly/[email protected]
# or
npm install @redocly/[email protected]
# or
npm install @redocly/[email protected] |
packages/core/src/rules/oas3/__tests__/spec-components-invalid-map-name.test.ts
Outdated
Show resolved
Hide resolved
cd7edd8 to
92eb22d
Compare
9d5fb3a to
a8d6ab4
Compare
|
c10a705 to
dda67fe
Compare
ba942f5 to
18dac92
Compare
2b6f9bd to
7447189
Compare
|
📦 A new experimental 🧪 version v0.0.0-snapshot.1766501159 of Redocly CLI has been published for testing. Install with NPM: npm install @redocly/[email protected]
# or
npm install @redocly/[email protected]
# or
npm install @redocly/[email protected] |
|
📦 A new experimental 🧪 version v0.0.0-snapshot.1766508919 of Redocly CLI has been published for testing. Install with NPM: npm install @redocly/[email protected]
# or
npm install @redocly/[email protected]
# or
npm install @redocly/[email protected] |
|
📦 A new experimental 🧪 version v0.0.0-snapshot.1766518457 of Redocly CLI has been published for testing. Install with NPM: npm install @redocly/[email protected]
# or
npm install @redocly/[email protected]
# or
npm install @redocly/[email protected] |
What/Why/How?
Fixed an issue where JSON Pointers containing special characters (like
%) were not properly URI-encoded when used as URI identifiers, causing validation errors with properties containing percent signs or other special characters.Additionally improved performance benchmarking by unifying the pipelines, making the run faster by linting only one API, and adding more versions to compare.
Reference
Related change => #170
Testing
Tested with Redoc/Realm/Reunite => https://github.com/Redocly/redocly/pull/19833
Screenshots (optional)
Check yourself
Security