-
Notifications
You must be signed in to change notification settings - Fork 5
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
Feature/voice2 #31
base: main
Are you sure you want to change the base?
Feature/voice2 #31
Conversation
docs: fix typos and update broken link in docs
chore: fix spelling issues
fix: DenyLoginSubtask
…message-add fix: twitter - add actions suppress action ability.
fix: rag optimizations/fixes for context.
…tel-plugin fix: quick-intel plugin: optimize template/add suppress init msg
…message-add fix: discord - add actions suppress action ability.
The error logs indicate that there are multiple issues occurring in the application:
To address these issues:
Example of implementing the keepalive: function startKeepAlive(websocket, sessionId) {
setInterval(() => {
const keepaliveMessage = {
janus: "keepalive",
session_id: sessionId
};
websocket.send(JSON.stringify(keepaliveMessage));
}, 30000); // Send keepalive every 30 seconds
} Call this function when the WebSocket connection opens, and ensure to clear the interval when destroying the connection. By addressing these issues, particularly the Janus session maintenance, you should be able to maintain a stable connection and prevent the ICE connection failures. Citations: |
|
Better
|
now we are testing just discord via default char in debug mode
Relates to
Risks
Background
What does this PR do?
What kind of change is this?
Documentation changes needed?
Testing
Where should a reviewer start?
Detailed testing steps