Skip to content

fix: encryption and other fixes#57

Merged
TEJASNARAYANS merged 8 commits into
mainfrom
fix-encryption
Mar 18, 2026
Merged

fix: encryption and other fixes#57
TEJASNARAYANS merged 8 commits into
mainfrom
fix-encryption

Conversation

@TEJASNARAYANS

@TEJASNARAYANS TEJASNARAYANS commented Mar 18, 2026

Copy link
Copy Markdown
Contributor

What Changed?

Encryption key persistence across CLI/Docker: Replaced Docker named volume with a shared .data/ directory bind mount so both CLI and Docker read/write the same encryption key file. New keys are saved to .data/.encryption_key with fallback to the legacy .encryption_key for backward compatibility.

Worker network_mode: host removal: Removed unnecessary network_mode: host from the worker service, which was preventing it from resolving Docker service names (db, redis) and causing DB connectivity failures. Both API and worker now share the same bridge network and config.docker.yml.

API key whitespace stripping: Added .strip() on both encrypt and decrypt paths to prevent accidental leading/trailing whitespace from corrupting API keys. This fixed a Vapi 400 "failed to extract key" error caused by a leading space in the stored API key.

Vapi error diagnostics: Improved retrieve_call_metrics to capture and log Vapi's actual error response body before raising, instead of only logging the HTTP status code. Also removed unnecessary Content-Type: application/json header from GET requests.

Frontend error surfacing: Updated frontend catch blocks for Retell, Vapi, and ElevenLabs to extract error.response.data.detail for toast notifications instead of showing generic Axios errors. Backend was also updated to pass through human-readable provider error messages (e.g., "Trial over quota, please add payment.").

Deleted redundant config: Removed config.docker.worker.yml (separate worker config with localhost URLs) since the worker no longer needs network_mode: host. Reverted app/config.py env var override logic that was only needed for the host networking workaround.

Release Label

Select one semantic version bump intent for this PR:

  • major - breaking change, next release bumps major version
  • minor - backward-compatible feature, next release bumps minor version
  • fix - backward-compatible bug fix, next release bumps patch version
  • No label (defaults to patch release)

If you do not have permission to apply labels, mention the intended release label here and a maintainer will set it.

Checklist

  • I have read the CONTRIBUTING.md guide.
  • My code follows the project's style guidelines.
  • I have added tests that prove my fix is effective or my feature works.
  • I have updated documentation where needed.

@TEJASNARAYANS TEJASNARAYANS added the fix Fix version release label Mar 18, 2026
@TEJASNARAYANS TEJASNARAYANS merged commit f2ce0f2 into main Mar 18, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

fix Fix version release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant