Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR refactors the encryption utilities and adds a new Telegram users feature. The changes include replacing the direct encryption functions with a new Cipher class that uses salt-based key derivation, and introducing a complete tg.users table with corresponding API endpoints.
- Replaces simple encrypt/decrypt functions with a salt-based
Cipherclass for improved security - Adds new
tg.userstable and router with encrypted user data storage - Updates version numbers across packages
Reviewed Changes
Copilot reviewed 13 out of 13 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| backend/src/utils/encrypt.ts | Removed old encryption utility functions |
| backend/src/utils/cipher.ts | Added new Cipher class with salt-based key derivation |
| backend/src/routers/tg/users.ts | Added new router for Telegram users with encryption/decryption |
| backend/src/routers/tg/messages.ts | Updated to use new Cipher class instead of direct encrypt/decrypt |
| backend/src/routers/tg/permissions.ts | Minor schema fix for error union type |
| backend/src/db/schema/tg/users.ts | Added database schema for users table |
| backend/drizzle/ | Database migration files for new users table |
| package.json files | Version bumps to 0.12.0 |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
There was a problem hiding this comment.
Pull Request Overview
Copilot reviewed 13 out of 13 changed files in this pull request and generated no new comments.
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
No description provided.