[PR #212] [MERGED] Fix writing inline keys with unicode chars #167

New issue

Closed

opened 2026-05-23 08:32:25 -06:00 by gitea-mirror · 0 comments

gitea-mirror commented

2026-05-23 08:32:25 -06:00

Owner

📋 Pull Request Information

Original PR: https://github.com/appy-one/acebase/pull/212
Author: @appy-one
Created: 4/3/2023
Status: ✅ Merged
Merged: 4/10/2023
Merged by: @appy-one

Base: master ← Head: fix/inline-unicode-keys

📝 Commits (1)

3d6a46c Fix writing inline keys with unicode chars

📊 Changes

1 file changed (+3 additions, -8 deletions)

View changed files

📝 src/storage/binary/index.ts (+3 -8)

📄 Description

Fixes the issue where keys with unicode characters are not correctly written to the db causing them to become unreadable. This only was able to happen if the internal Key Index Table was full, or if IPC was used in clustering mode. This will not fix existing data, but does prevent it from happening in the future. If you have a broken db because of this issue and want it fixed, let me know. Fixes #209

_{🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.}

## 📋 Pull Request Information **Original PR:** https://github.com/appy-one/acebase/pull/212 **Author:** [@appy-one](https://github.com/appy-one) **Created:** 4/3/2023 **Status:** ✅ Merged **Merged:** 4/10/2023 **Merged by:** [@appy-one](https://github.com/appy-one) **Base:** `master` ← **Head:** `fix/inline-unicode-keys` --- ### 📝 Commits (1) - [`3d6a46c`](https://github.com/appy-one/acebase/commit/3d6a46c49ea940fc10dc51a9f2444595ce63ccce) Fix writing inline keys with unicode chars ### 📊 Changes **1 file changed** (+3 additions, -8 deletions) <details> <summary>View changed files</summary> 📝 `src/storage/binary/index.ts` (+3 -8) </details> ### 📄 Description Fixes the issue where keys with unicode characters are not correctly written to the db causing them to become unreadable. This only was able to happen if the internal Key Index Table was full, or if IPC was used in clustering mode. This will not fix existing data, but does prevent it from happening in the future. If you have a broken db because of this issue and want it fixed, let me know. Fixes #209 --- <sub>🔄 This issue represents a GitHub Pull Request. It cannot be merged through Gitea due to API limitations.</sub>