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 guide for screego #1800

Merged
merged 11 commits into from
Oct 24, 2024
Merged

Add guide for screego #1800

merged 11 commits into from
Oct 24, 2024

Conversation

no-one
Copy link
Contributor

@no-one no-one commented Sep 4, 2024

Fixes #1799

@no-one
Copy link
Contributor Author

no-one commented Oct 8, 2024

A question for the person who will review this pull request later: I have not currently entered a port range for the TURN server. There is the following example in the config:

# Limit the ports that TURN will use for data relaying.
# Format: min:max
# Example:
#   50000:55000
SCREEGO_TURN_PORT_RANGE=

In most cases this will probably not be so relevant (because STUN can be used instead of TURN) but if this is not the case: As you can see, a fairly large range is specified. However, ports can only be opened individually. What do you suggest?

@nichtmax
Copy link
Member

A question for the person who will review this pull request later: I have not currently entered a port range for the TURN server. There is the following example in the config:

# Limit the ports that TURN will use for data relaying.
# Format: min:max
# Example:
#   50000:55000
SCREEGO_TURN_PORT_RANGE=

In most cases this will probably not be so relevant (because STUN can be used instead of TURN) but if this is not the case: As you can see, a fairly large range is specified. However, ports can only be opened individually. What do you suggest?

I wouldn't bother. The documentation states: "While STUN should work for most cases, there are stricter NATs f.ex. Symmetric NATs where it doesn’t, then, TURN will be used."

How about an info box with a link to their documentation? 5000 Ports is a bit excessive, I don't know what's the minimum range, maybe ask them? ☺️

source/guide_screego.rst Outdated Show resolved Hide resolved
@nichtmax nichtmax marked this pull request as draft October 10, 2024 07:26
@nichtmax nichtmax self-assigned this Oct 10, 2024
Copy link
Member

@nichtmax nichtmax left a comment

Choose a reason for hiding this comment

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

Thanks for your guide! I left you some comments, I'll test everything when these are resolved.

@nichtmax nichtmax removed their assignment Oct 10, 2024
Copy link
Member

@nichtmax nichtmax left a comment

Choose a reason for hiding this comment

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

One small thing left ☺️

source/guide_screego.rst Outdated Show resolved Hide resolved
@nichtmax nichtmax marked this pull request as draft October 23, 2024 07:59
@no-one no-one marked this pull request as ready for review October 23, 2024 17:28
@no-one no-one requested a review from nichtmax October 23, 2024 17:28
@nichtmax
Copy link
Member

Thanks! Please get in touch via [email protected] for your goodie!

@nichtmax nichtmax merged commit b95d701 into Uberspace:main Oct 24, 2024
4 checks passed
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.

[screego] add guide
2 participants