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

Modify default INTEGRATED_TURN_LISTEN_IP #43

Merged
merged 1 commit into from May 25, 2023

Conversation

Rados13
Copy link
Contributor

@Rados13 Rados13 commented May 24, 2023

No description provided.

@Rados13 Rados13 requested review from mickel8 and LVala May 24, 2023 11:29
@codecov
Copy link

codecov bot commented May 24, 2023

Codecov Report

Merging #43 (47dec54) into main (744552b) will decrease coverage by 0.80%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##             main      #43      +/-   ##
==========================================
- Coverage   90.21%   89.41%   -0.80%     
==========================================
  Files          32       32              
  Lines         378      378              
==========================================
- Hits          341      338       -3     
- Misses         37       40       +3     

see 1 file with indirect coverage changes


Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 744552b...47dec54. Read the comment docs.

@@ -13,7 +13,7 @@ It leverages the [Membrane RTC Engine](https://github.com/jellyfish-dev/membrane

There are two ways of running Jellyfish:
- building from source (requires Elixir and native dependencies)
- using Jellyfish Docker images
- using Jellyfish Docker images (Remember that you should set `INTEGRATED_TURN_IP` env to your public IP address and `INTEGRATED_TURN_LISTEN_IP` to `0.0.0.0`).
Copy link
Contributor

Choose a reason for hiding this comment

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

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think it could stay here, but I also added it to the jellyfish docs.
fishjam-dev/fishjam-docs#12

Copy link
Member

@LVala LVala left a comment

Choose a reason for hiding this comment

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

👍

@Rados13 Rados13 merged commit 7ac3fd3 into main May 25, 2023
5 checks passed
@Rados13 Rados13 deleted the RTC-262-no-media-transferred branch May 25, 2023 07:30
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.

None yet

3 participants