ilib-localedata: Fix race condition in LocaleData.ensureLocale()#148
Open
ehoogerbeets wants to merge 557 commits intocvtLocaleDataToJestfrom
Open
ilib-localedata: Fix race condition in LocaleData.ensureLocale()#148ehoogerbeets wants to merge 557 commits intocvtLocaleDataToJestfrom
ehoogerbeets wants to merge 557 commits intocvtLocaleDataToJestfrom
Conversation
🦋 Changeset detectedLatest commit: 6819925 The changes in this PR will be included in the next version bump. This PR includes changesets to release 24 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 |
Contributor
Jest Code CoverageCode Coverage for changed files • (0%)
|
ehoogerbeets
force-pushed
the
cvtLocaleDataToJest
branch
from
d8cc3a8 to
b467ecb
Compare
ehoogerbeets
force-pushed
the
fixEnsureLocale
branch
from
eb5024a to
30ab4f9
Compare
ehoogerbeets
force-pushed
the
fixEnsureLocale
branch
from
30ab4f9 to
7d9b1e1
Compare
ehoogerbeets
force-pushed
the
fixEnsureLocale
branch
from
ee1a5f3 to
c9488d8
Compare
ehoogerbeets
force-pushed
the
cvtLocaleDataToJest
branch
from
b467ecb to
fe30e85
Compare
ehoogerbeets
changed the title
(ilib-loctool-pendo-md) Rework how markdown syntax is escaped
Version Packages
ilib-lint: Fix a bug in the state checker rule
- it is not used very much in Japanese in translations of English sentences that end in an exclamation point, so we don't need to check it
…izeOnly is false or pseudo is enabled `translations of Project' is not used when localizeOnly is true or pseudo is disabled. It is referenced by PseudoHant and not used in `write` step.
- if the config includes mappings like "**/*.mdx", then the file name extensions in that mapping should be added to the list of valid extensions that this plugin handles on the assumption that the user put those mappings there on purpose to pick up that type of file.
- loctool now warns when the mappings mention an extension that is not mentioned in extensions
Version Packages
…rings set When a mapping uses a frontmatter allowlist (array of key names), the extraction phase correctly filters to only extract those keys. However, during localization the yaml plugin processes all frontmatter keys and reports any without a translation as 'new'. Keys outside the allowlist (e.g. 'type', 'mode', 'related_pages') were never sent to the TMS, so they will never have translations — yet they appeared in the new-strings XLIFF as false positives. Filter yaml new-resources in MdxFileType.getNew() using the same frontmatter allowlist that governs extraction. This ensures symmetry between what is extracted and what is reported as untranslated. When frontmatter is boolean true (localize all), no filtering is applied.
fix(ilib-loctool-mdx): filter non-allowlisted frontmatter
ilib-loctool-mdx: Add missing changeset for frontmatter filter fix
Version Packages
ilib-loctool-openapi: Move open api plugin from ilib-js/ilib-loctool-openapi to here
- regex was not anchored to the beginning of the string, so when you have a template like "[language]/[dir]/[filename]", it would match against "guides/ai-zone/index.mdx" where the language would end up being "des". (Last 3 letters of "guides".) - Now with it anchored properly, it returns "" for the language because "guides" does not match an ISO language code.
Fix a problem with getLocaleFromPath
Version Packages
ilib-assemble: Fix the issue of duplicated code inclusion of legacyilibassemble
Add key filtering to prevent prototype pollution attacks (CWE-1321) in JSUtils.extend, extend2, merge, deepCopy, and shallowCopy functions. The fix blocks __proto__, constructor, and prototype keys from being processed during object merging/extending operations, preventing attackers from polluting Object.prototype via malicious input. Fixes #266
If there was anyOf or oneOf in the schema, it would sometimes localize the json object as a string instead of as an object.
fix(ilib-common): prevent prototype pollution
ilib-loctool-json: Fix localized output path and fix broken output
Version Packages
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
7 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.