Skip to content

Conversation

@erangi-ar
Copy link
Collaborator

No description provided.

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Copilot encountered an error and was unable to review this pull request. You can try again by re-requesting a review.

README.md Outdated
- Models searchable via dropdown with cache-enabled indicators.

- **Enhanced Security with RSA Encryption**
- LLM credentials encrypted with RSA-4096 asymmetric encryption before storage.
Copy link

Copilot AI Jan 14, 2026

Choose a reason for hiding this comment

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

Documentation claims RSA-4096 encryption, but the implementation in vault-init.sh line 195 uses RSA-2048 (openssl genrsa -out \"$TEMP_KEY_DIR/private.pem\" 2048). Update the documentation to accurately reflect RSA-2048.

Suggested change
- LLM credentials encrypted with RSA-4096 asymmetric encryption before storage.
- LLM credentials encrypted with RSA-2048 asymmetric encryption before storage.

Copilot uses AI. Check for mistakes.
Copy link
Collaborator

Choose a reason for hiding this comment

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

please address this

@erangi-ar erangi-ar marked this pull request as ready for review January 14, 2026 09:18
README.md Outdated
- Models searchable via dropdown with cache-enabled indicators.

- **Enhanced Security with RSA Encryption**
- LLM credentials encrypted with RSA-4096 asymmetric encryption before storage.
Copy link
Collaborator

Choose a reason for hiding this comment

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

please address this

@Thirunayan22 Thirunayan22 merged commit d5e5c51 into buerokratt:wip Jan 14, 2026
4 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants