Skip to content

Conversation

@randolf
Copy link

@randolf randolf commented Mar 12, 2022

Added instructions for adding exceptions to the "nftables" firewall.

Added instructions for adding exceptions to the "nftables" firewall.
If you are using `nftables`, you can add the following line to `/etc/nftables.conf` in the "input" chain, which is in the "inet filter" table:

```
tcp dport { 80, 443, 10000, 22, 3478, 5349 } ct state new accept
Copy link
Member

Choose a reason for hiding this comment

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

Port 5349 is no longer necessary, but 5222 is.

Copy link
Author

Choose a reason for hiding this comment

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

TCP port 5349 is in the documentation. Shall I update the documentation too (changing port 5349 to 5222)?

Copy link
Member

Choose a reason for hiding this comment

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

Copy link
Member

Choose a reason for hiding this comment

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

Ops, my bad I read too quickly and mixed it up with the XMPP component port.

Copy link
Member

Choose a reason for hiding this comment

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

We still need 5222 though right?

Copy link
Member

Choose a reason for hiding this comment

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

From the outside no, it is used only from jvb, jibri and maybe jigasi(jiigasi can use bosh though and port 443), and normally that will be only the internal network for prosody.

Copy link
Member

Choose a reason for hiding this comment

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

Generally speaking yeah but if one needs to scale to multiple JVBs they would so I think we better mention it early WDYT?

Update the nftables firewall with:

```
sudo /etc/nftables.conf
Copy link
Member

Choose a reason for hiding this comment

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

This line will not do anything

@saghul
Copy link
Member

saghul commented Mar 25, 2022

Ping?

@RudraSen2
Copy link
Contributor

Ping

@saghul
Copy link
Member

saghul commented Jul 6, 2022

Waiting on the requested changes to be made.

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.

4 participants