Skip to content

Fix critical runtime bugs and security issues#78

Merged
niralbhalodia merged 2 commits intodevelopfrom
critical_issues_solved
Mar 11, 2026
Merged

Fix critical runtime bugs and security issues#78
niralbhalodia merged 2 commits intodevelopfrom
critical_issues_solved

Conversation

@harshdesaii
Copy link
Collaborator

Description

Fix frozen timestamps, recursive retry risk, async/sync memory mismatch, replace fake embeddings with real computation and add guidance for secure JWT configuration.

Fixes # (issue)

  • Fixed datetime.now() frozen at import time
  • Replaced asdict(self) with self.model_dump()
  • Fixed recursive retry in router
  • Replaced dummy zero embeddings with real pooled embeddings
  • Fixed claude image payload encoding(hex -> base64)
  • Hardcoded JWT secret/default users removed; env-based JWT setup enforced
  • Fixed async/sync mismatch in memory layer

@niralbhalodia niralbhalodia merged commit f791d83 into develop Mar 11, 2026
1 check 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.

2 participants