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

Firewall issue no longer happening #1835

Merged
merged 1 commit into from
Oct 9, 2024

Conversation

yanokwa
Copy link
Member

@yanokwa yanokwa commented Jul 15, 2024

With an existing install, I can upload a form, preview it, generate a public access link, send in a submission with that link, edit the submission, and it all seems to work. Maybe it’d break with a fresh install, but doubt it. I think we can proceed with removing this warning from the docs.

Our docs say we require Docker v23, but this was Docker v27. Doubt it breaks on v23.

root@test:~# lsb_release -a
No LSB modules are available.
Distributor ID:	Ubuntu
Description:	Ubuntu 20.04.6 LTS
Release:	20.04
Codename:	focal
root@test:~#

root@test:~# docker --version && docker compose version
Docker version 27.0.3, build 7d4bcd8
Docker Compose version v2.28.1

root@test:~# ufw status
Status: active

To                         Action      From
--                         ------      ----
22/tcp                     LIMIT       Anywhere
80                         ALLOW       Anywhere
443                        ALLOW       Anywhere
22/tcp (v6)                LIMIT       Anywhere (v6)
80 (v6)                    ALLOW       Anywhere (v6)
443 (v6)                   ALLOW       Anywhere (v6)

@yanokwa yanokwa requested a review from lognaturel July 15, 2024 17:38

The goal here is to ensure that it is possible to access the host through its external IP from within each Docker container. In particular, if you can successfully ``curl`` your Central website over HTTPS on its public domain name, all Enketo features should work correctly.

Getting and Setting Up Central
Copy link
Member

Choose a reason for hiding this comment

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

Keep this header

Copy link
Member Author

Choose a reason for hiding this comment

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

The header is up on line 92

@yanokwa yanokwa merged commit 8189771 into getodk:master Oct 9, 2024
1 check passed
@yanokwa yanokwa deleted the remove-firewall-warning branch October 9, 2024 20:52
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.

2 participants