Skip to content
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

adjust talk hpb config #402

Merged
merged 1 commit into from
Mar 18, 2022
Merged

adjust talk hpb config #402

merged 1 commit into from
Mar 18, 2022

Conversation

szaimen
Copy link
Collaborator

@szaimen szaimen commented Mar 18, 2022

Close #400

Signed-off-by: szaimen szaimen@e.mail.de

@szaimen szaimen added 3. to review Waiting for reviews enhancement New feature or request labels Mar 18, 2022
@szaimen szaimen added this to the next milestone Mar 18, 2022
@szaimen
Copy link
Collaborator Author

szaimen commented Mar 18, 2022

@muellerm01 does this look good to you?

@szaimen
Copy link
Collaborator Author

szaimen commented Mar 18, 2022

@morph027 it looks like the hpb config might not apply anymore with new releases of the signaling server. See
https://github.com/strukturag/nextcloud-spreed-signaling/blob/master/server.conf.in
vs

cat << SIGNALING_CONF > "/etc/signaling/server.conf"
[http]
listen = 0.0.0.0:8081
[app]
debug = false
[sessions]
hashkey = $(openssl rand -hex 16)
blockkey = $(openssl rand -hex 16)
[clients]
internalsecret = $(openssl rand -hex 16)
[backend]
allowed = ${NC_DOMAIN}
allowall = false
secret = ${SIGNALING_SECRET}
timeout = 10
connectionsperhost = 8
[nats]
url = nats://127.0.0.1:4222
[mcu]
type = janus
url = ws://127.0.0.1:8188
[turn]
apikey = ${JANUS_API_KEY}
secret = ${TURN_SECRET}
servers = turn:$NC_DOMAIN:3478?transport=tcp,turn:$NC_DOMAIN:3478?transport=udp
SIGNALING_CONF

Do you maybe have some ideas/pointers what we need to change?

@morph027
Copy link

Are there any error messages when starting the container?

@szaimen
Copy link
Collaborator Author

szaimen commented Mar 18, 2022

@morph027 here is a logfile: #400 (comment)

@szaimen
Copy link
Collaborator Author

szaimen commented Mar 18, 2022

Thanks for having a look! :)

Signed-off-by: szaimen <szaimen@e.mail.de>
@szaimen szaimen force-pushed the enh/400/increase-talk-hpb-limit branch from c7b843e to 52c926e Compare March 18, 2022 15:13
@szaimen szaimen changed the title increase talk hpb limit per host adjust talk hpb config Mar 18, 2022
@szaimen szaimen merged commit 9c64253 into main Mar 18, 2022
@delete-merged-branch delete-merged-branch bot deleted the enh/400/increase-talk-hpb-limit branch March 18, 2022 15:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Increase Nextcloud HPB limit?
2 participants