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

[ADD] SFU server #1

Merged
merged 1 commit into from
Dec 5, 2023
Merged

[ADD] SFU server #1

merged 1 commit into from
Dec 5, 2023

Conversation

ThanhDodeurOdoo
Copy link
Collaborator

The SFU is a server used to relay streaming between the participants of a call to provide better scalability and performance than the current peer-to-peer connections.

README
Old PR: https://github.com/odoo/enterprise/pull/45861
Related to: odoo/odoo#132153

task-2765922

The SFU is a server used to relay streaming between the participants of
a call to provide better scalability and performance than the current
peer-to-peer connections.

[README](https://github.com/odoo/sfu/tree/master-discuss-rtc-server2-tso/)
Old PR: odoo/enterprise#45861
Related to: odoo/odoo#132153

task-2765922
@ThanhDodeurOdoo ThanhDodeurOdoo force-pushed the master-discuss-rtc-server2-tso branch from 209af28 to 0c61202 Compare December 5, 2023 12:50
Comment on lines +9 to +10
"node": true,
"browser": false,
Copy link
Collaborator

Choose a reason for hiding this comment

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

isn't node or browser differently true or false (or sometimes both at the same time) depending on the file?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

the overrides below are setting the env with the right values for the few files where it is not (just) node

README.md Show resolved Hide resolved

- GET `/v1/stats`: returns the server statistics in JSON:
```json
[ // one array entry per channel
Copy link
Collaborator

@seb-odoo seb-odoo Dec 5, 2023

Choose a reason for hiding this comment

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

Comment is invalid json, suggestion to use a specific key "_comment" or comment it separately

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

fix for both README comments in #2

Copy link
Collaborator

@seb-odoo seb-odoo left a comment

Choose a reason for hiding this comment

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

I think we reached the point where we should just merge and fix if necessary. Keeping the full diff pending hoping to reach perfection becomes counter-productive now.

@seb-odoo seb-odoo merged commit 5c3c459 into master Dec 5, 2023
@nim-odoo nim-odoo deleted the master-discuss-rtc-server2-tso branch December 5, 2023 15:36
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