Skip to content

refactor: 更换加密方法&迁移配置项#396

Merged
tangge233 merged 8 commits intodevfrom
feat/Encrypt
Apr 13, 2025
Merged

refactor: 更换加密方法&迁移配置项#396
tangge233 merged 8 commits intodevfrom
feat/Encrypt

Conversation

@tangge233
Copy link
Copy Markdown
Contributor

@tangge233 tangge233 commented Apr 5, 2025

我也不是这个领域的大佬,如果有问题请及时提出

Close #77

@tangge233 tangge233 added the 🕑 等待合并 已处理完毕,正在等待代码合并入主分支 label Apr 5, 2025
@tangge233 tangge233 added this to the 2.10.6 milestone Apr 5, 2025
@tangge233 tangge233 requested a review from a team April 5, 2025 08:03
@EncVar
Copy link
Copy Markdown
Contributor

EncVar commented Apr 5, 2025

换成获取主板序列号会不会好一些(考虑到有更换CPU的可能性

@EncVar EncVar added 🚧 正在处理 开发人员正在对该内容进行开发、测试或修复,进展中 and removed 🕑 等待合并 已处理完毕,正在等待代码合并入主分支 labels Apr 5, 2025
@Pigeon0v0
Copy link
Copy Markdown
Contributor

都迁移了直接丢 %appdata%\.pclce\config.json 吧 = =

或者 toml 啥的也行,看怎么顺怎么来

@wyc-26
Copy link
Copy Markdown
Contributor

wyc-26 commented Apr 6, 2025

换成获取主板序列号会不会好一些(考虑到有更换CPU的可能性

那我换主板呢(

@Hill23333
Copy link
Copy Markdown
Contributor

换cpu和换主板都是小概率事件,都可以

@tangge233 tangge233 marked this pull request as draft April 8, 2025 15:44
@SALTWOOD
Copy link
Copy Markdown
Member

SALTWOOD commented Apr 8, 2025

有个小疑问……这个 PR 是不是相当于把注册表的一堆项目全部文件化了?

@pnpn111
Copy link
Copy Markdown

pnpn111 commented Apr 9, 2025

主板序列号应该比CPU的好,这个换的概率应该会更小一点点。
另外请问一下CE版真的有加密数据的必要吗,最多用Base64编码一下就够了吧?

@whitecat346
Copy link
Copy Markdown
Member

主板序列号应该比CPU的好,这个换的概率应该会更小一点点。 另外请问一下CE版真的有加密数据的必要吗,最多用Base64编码一下就够了吧?

最好还是加密一下的,直接明文存储敏感数据实在是对用户不尊重

@tangge233 tangge233 marked this pull request as ready for review April 12, 2025 17:38
Copy link
Copy Markdown
Member

@SALTWOOD SALTWOOD left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

仔细看了看,没发现明显的问题

@tangge233 tangge233 changed the title feat: 更换加密方法 feat: 更换加密方法&迁移配置项 Apr 12, 2025
Copy link
Copy Markdown
Contributor

@Pigeon0v0 Pigeon0v0 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@tangge233 tangge233 requested review from Pigeon0v0 and SALTWOOD April 13, 2025 03:33
@tangge233 tangge233 changed the title feat: 更换加密方法&迁移配置项 refactor: 更换加密方法&迁移配置项 Apr 13, 2025
@Pigeon0v0
Copy link
Copy Markdown
Contributor

所以迁移完以后注册表还会留个项吗...?

@tangge233
Copy link
Copy Markdown
Contributor Author

所以迁移完以后注册表还会留个项吗...?

每次读取时自动迁移,迁移后会删除注册表里对应的项。如果发现已经被迁移的项,说明用户使用过高版本后再次使用过低版本 CE,不选择迁移上次产生的数据

@tangge233 tangge233 merged commit 4ca72cc into dev Apr 13, 2025
4 checks passed
@tangge233 tangge233 deleted the feat/Encrypt branch April 13, 2025 04:48
@PCL-Community-Bot PCL-Community-Bot added 👌 完成 相关问题已修复或功能已实现,计划在下次版本更新时正式上线 and removed 🚧 正在处理 开发人员正在对该内容进行开发、测试或修复,进展中 labels Apr 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

👌 完成 相关问题已修复或功能已实现,计划在下次版本更新时正式上线

Projects

None yet

Development

Successfully merging this pull request may close these issues.

迁移设备端加密数据到 AppData

9 participants