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

Janus API 15 < 16 Build #97

Closed
calvincs opened this issue Feb 8, 2022 · 4 comments
Closed

Janus API 15 < 16 Build #97

calvincs opened this issue Feb 8, 2022 · 4 comments

Comments

@calvincs
Copy link

calvincs commented Feb 8, 2022

After building the plugin and installing, I am receiving the following message after starting Janus with the plugin:

Plugins folder: /opt/janus/lib/janus/plugins
Loading plugin 'libjanus_plugin_sfu.so'...
Sessions watchdog started
Joining Janus requests handler thread
[ERR] [janus.c:main:5471] The 'janus.plugin.sfu' plugin was compiled against an older version of the API (15 < 16), skipping it: update it to enable it again

Current Janus server build:

root@janus:~/janus-plugin-sfu# /opt/janus/bin/janus -v
Janus commit: 7c83c093e05400d3c18e0cac08f1b797e35dfe4f
Compiled on:  Mon Feb  7 04:15:10 UTC 2022

Is this plugin being maintained anymore, will this be updated?

Thanks in advance!

@vincentfretin
Copy link
Contributor

Hi @calvincs see networked-aframe/naf-janus-adapter#23

@calvincs
Copy link
Author

calvincs commented Feb 9, 2022

Awesome, thanks @vincentfretin
So we are just waiting for things to merge and we should be good to go then?
Assuming I read those threads correctly.

@vincentfretin
Copy link
Contributor

You have several possibilities, you can use an older janus-gateway version like v0.11.1 that was properly working, or use the latest 0.x branch but revert the janus-gateway commit that introduced the API change like I explained in the PR, or wait that I do the forks of janus-plugin-rs and janus-plugin-sfu and do the required changes in a new PR for those two packages, but no ETA for that. You can also just follow the install instructions with the changes in my PR with my janus-gateway fork where I already reverted the commit.

@calvincs
Copy link
Author

Sounds great! Thanks @vincentfretin I will close this out and keep an eye on progress.
It's my hope that I and others can help you maintain these components in the future. Anything we can do, please let us know.

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

No branches or pull requests

2 participants