Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: add encryption for sensitive data and configure environment variables #245

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Snox-dev1
Copy link

@Snox-dev1 Snox-dev1 commented Dec 3, 2024

📋 Description

  • Introduced a new .env.exemple file containing a sample encryption key.
  • Updated server/index.js to load environment variables using dotenv.
  • Enhanced the Identity model to include encryption and decryption hooks for sensitive fields (password, SSH key, passphrase).
  • Added a new encryption.js utility for handling encryption and decryption using AES-256-GCM.

🚀 Changes made to ...

  • 🔧 Server
  • 🖥️ Client
  • 📚 Documentation
  • 🔄 Other: env and readme

✅ Checklist

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have looked for similar pull requests in the repository and found none

🔗 Related Issues #216

Fixes #216
Update readme to add : #101

…iables

- Introduced a new `.env.exemple` file containing a sample encryption key.
- Updated `server/index.js` to load environment variables using `dotenv`.
- Enhanced the `Identity` model to include encryption and decryption hooks for sensitive fields (password, SSH key, passphrase).
- Added a new `encryption.js` utility for handling encryption and decryption using AES-256-GCM.
…instructions

- Added SERVER_PORT and NODE_ENV variables to .env.exemple for better configuration.
- Expanded README to include detailed development setup instructions, prerequisites, and configuration options for the server.
Copy link

@luowei luowei left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants