Merge commit from fork

* fix: prototype pollution in `handleFlatJson`

* chore: add npm provenance

* chore: add permissions for github actions

Author: kazupon

.github/workflows/release.yml

@@ -8,6 +8,9 @@ on:
       - 'v*'
 env:
   NODE_OPTIONS: --max-old-space-size=6144
+permissions:
+  id-token: write
+  contents: write
 
 jobs:
   release:
@@ -59,3 +62,4 @@ jobs:
           ./scripts/release.sh
         env:
           NPM_AUTH_TOKEN: ${{secrets.NPM_AUTH_TOKEN}}
+          NPM_CONFIG_PROVENANCE: true

packages/vue-i18n-core/src/utils.ts

@@ -77,6 +77,9 @@ export function handleFlatJson(obj: unknown): unknown {
       let currentObj = obj
       let hasStringValue = false
       for (let i = 0; i < lastIndex; i++) {
+        if (subKeys[i] === '__proto__') {
+          throw new Error(`unsafe key: ${subKeys[i]}`)
+        }
         if (!(subKeys[i] in currentObj)) {
           currentObj[subKeys[i]] = create()
         }

packages/vue-i18n-core/test/utils.test.ts

@@ -10,56 +10,71 @@ vi.mock('@intlify/shared', async () => {
 import { handleFlatJson } from '../src/utils'
 import { I18nWarnCodes, getWarnMessage } from '../src/warnings'
 
-test('handleFlatJson', () => {
-  const mockWarn = vi.spyOn(shared, 'warn')
-  // eslint-disable-next-line @typescript-eslint/no-empty-function
-  mockWarn.mockImplementation(() => {})
+describe('handleFlatJson', () => {
+  test('basic', () => {
+    const mockWarn = vi.spyOn(shared, 'warn')
+    // eslint-disable-next-line @typescript-eslint/no-empty-function
+    mockWarn.mockImplementation(() => {})
 
-  const obj = {
-    a: { a1: 'a1.value' },
-    'a.a2': 'a.a2.value',
-    'b.x': {
-      'b1.x': 'b1.x.value',
-      'b2.x': ['b2.x.value0', 'b2.x.value1'],
-      'b3.x': { 'b3.x': 'b3.x.value' }
-    },
-    c: {
-      'animal.dog': 'Dog',
-      animal: 'Animal'
-    },
-    d: {
-      'animal.dog': 'Dog',
-      animal: {}
-    }
-  }
-  const expectObj = {
-    a: {
-      a1: 'a1.value',
-      a2: 'a.a2.value'
-    },
-    b: {
-      x: {
-        b1: { x: 'b1.x.value' },
-        b2: { x: ['b2.x.value0', 'b2.x.value1'] },
-        b3: { x: { b3: { x: 'b3.x.value' } } }
+    const obj = {
+      a: { a1: 'a1.value' },
+      'a.a2': 'a.a2.value',
+      'b.x': {
+        'b1.x': 'b1.x.value',
+        'b2.x': ['b2.x.value0', 'b2.x.value1'],
+        'b3.x': { 'b3.x': 'b3.x.value' }
+      },
+      c: {
+        'animal.dog': 'Dog',
+        animal: 'Animal'
+      },
+      d: {
+        'animal.dog': 'Dog',
+        animal: {}
       }
-    },
-    c: {
-      'animal.dog': 'Dog',
-      animal: 'Animal'
-    },
-    d: {
-      animal: {
-        dog: 'Dog'
+    }
+    const expectObj = {
+      a: {
+        a1: 'a1.value',
+        a2: 'a.a2.value'
+      },
+      b: {
+        x: {
+          b1: { x: 'b1.x.value' },
+          b2: { x: ['b2.x.value0', 'b2.x.value1'] },
+          b3: { x: { b3: { x: 'b3.x.value' } } }
+        }
+      },
+      c: {
+        'animal.dog': 'Dog',
+        animal: 'Animal'
+      },
+      d: {
+        animal: {
+          dog: 'Dog'
+        }
       }
     }
-  }
 
-  expect(handleFlatJson(obj)).toEqual(expectObj)
-  expect(mockWarn).toHaveBeenCalled()
-  expect(mockWarn.mock.calls[0][0]).toEqual(
-    getWarnMessage(I18nWarnCodes.IGNORE_OBJ_FLATTEN, {
-      key: 'animal'
-    })
-  )
+    expect(handleFlatJson(obj)).toEqual(expectObj)
+    expect(mockWarn).toHaveBeenCalled()
+    expect(mockWarn.mock.calls[0][0]).toEqual(
+      getWarnMessage(I18nWarnCodes.IGNORE_OBJ_FLATTEN, {
+        key: 'animal'
+      })
+    )
+  })
+
+  // security advisories
+  // ref: https://github.com/intlify/vue-i18n/security/advisories/GHSA-p2ph-7g93-hw3m
+  test('prototype pollution', () => {
+    expect(() =>
+      handleFlatJson({ '__proto__.pollutedKey': 'pollutedValue' })
+    ).toThrow()
+    // @ts-ignore -- test
+    // eslint-disable-next-line no-proto
+    expect({}.__proto__.pollutedKey).toBeUndefined()
+    // @ts-ignore -- test
+    expect(Object.prototype.pollutedKey).toBeUndefined()
+  })
 })