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 IP filtering whitelist #574

Open
tomkralidis opened this issue Nov 14, 2023 · 4 comments
Open

add IP filtering whitelist #574

tomkralidis opened this issue Nov 14, 2023 · 4 comments
Assignees
Labels
enhancement New feature or request

Comments

@tomkralidis
Copy link
Collaborator

W2AT 2023-11-14:

We could add an include in nginx setup for IP whitelisting (allow/deny), for example.

@tomkralidis tomkralidis added the enhancement New feature or request label Nov 14, 2023
@tomkralidis tomkralidis added this to the sprint-013 milestone Nov 14, 2023
@maaikelimper
Copy link
Collaborator

We would also have to support the whitelisting for the MQTT broker in that case , right ?

Normally I would expect users to configure the IP restrictions upstream, like in the security group of the cloud-hosted instance or in the firewall of the network hosting their server.
Do you see any use-case whereby a user would want apply the IP restrictions at the level of the wis2box-services ?

@tomkralidis
Copy link
Collaborator Author

The use case for application level IP filtering supports the case where the deployment does not have options to have this set at the firewall/cloud level.

@kurt-hectic
Copy link

I think that most security conscious operators (likely those requiring IP filtering), will not implement fire-walling as part of the application stack that is protected, but with a dedicated firewall.
For example the AWS-borne wis2boxes would implement this by implementing a AWS Security Group, separating concerns of application and protection.

@maaikelimper maaikelimper removed this from the sprint-013 milestone Dec 13, 2023
@maaikelimper maaikelimper assigned efucile and unassigned maaikelimper Dec 13, 2023
@maaikelimper
Copy link
Collaborator

enrico to decided if/when we want to implement this

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants