Skip to content

fix(examples): update e2ee.py to use encryption kwarg and env var#5469

Merged
davidzhao merged 1 commit intomainfrom
chana/update-metrics
Apr 17, 2026
Merged

fix(examples): update e2ee.py to use encryption kwarg and env var#5469
davidzhao merged 1 commit intomainfrom
chana/update-metrics

Conversation

@aryeila
Copy link
Copy Markdown
Contributor

@aryeila aryeila commented Apr 16, 2026

Summary

Resolves DOCS-1278

🤖 Generated with Claude Code

- Rename `e2ee=` to `encryption=` on ctx.connect to match the renamed
  kwarg from #5454
- Load shared key from LIVEKIT_E2EE_KEY env var instead of hardcoding

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@aryeila aryeila marked this pull request as ready for review April 16, 2026 18:40
@chenghao-mou chenghao-mou requested a review from a team April 16, 2026 18:40
Copy link
Copy Markdown
Contributor

@devin-ai-integration devin-ai-integration Bot left a comment

Choose a reason for hiding this comment

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

✅ Devin Review: No Issues Found

Devin Review analyzed this PR and found no potential bugs to report.

View in Devin Review to see 1 additional finding.

Open in Devin Review

Copy link
Copy Markdown
Member

@davidzhao davidzhao left a comment

Choose a reason for hiding this comment

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

thanks @aryeila !

@davidzhao davidzhao merged commit e11127a into main Apr 17, 2026
28 checks passed
@davidzhao davidzhao deleted the chana/update-metrics branch April 17, 2026 05:38
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