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

fix: support other port than 443 #259

Merged
merged 2 commits into from
May 27, 2022

Conversation

delcroip
Copy link
Contributor

No description provided.

@issa-tseng
Copy link
Contributor

hey @delcroip:

we're all a little busy over here, but we'll take a look at this soon. thanks for sending this in.
it's looking okay to me at a first glance, but the port stuff gets complicated and passed into a bunch of places so it'll take a minute just to work through it all.

curious, what have you verified these changes by?

@delcroip
Copy link
Contributor Author

delcroip commented Nov 23, 2021

Hi,

l deployed it on my windows laptop on port 2443 (443 was already taken) in order to test forms while being in the plane, you can find the details of the config used there https://forum.getodk.org/t/could-not-connect-with-form-server-preview/35789/6?u=delcroip

It works fine since then

br

@yanokwa
Copy link
Member

yanokwa commented May 24, 2022

I have confirmed the following scenarios with a custom SSL cert and HTTPS port 8443:

  • The password reset email that is sent has the port information
  • Form downloads and submissions sending in Collect works
  • Previews and submission sending in Enketo works
  • CSV downloads and OData feed have submission data
  • Rebuild with LetsEncrypt on port 443 works and front page loads
  • Rebuild with custom SSL on port 8444 (different port than original) works and front page loads
    • New app user Collect submissions work, old ones do not.
    • New Enketo previews work, old ones do not.

The one issue I see is:

Another thing to consider is what happens if someone wants to run HTTP on something besides port 80. That is they want to go to http://domain.com:8081 and redirect to https://domain.com:8443. How do we (or do we) support that.

I discussed the above with @lognaturel and her bias is to merge and address the issue I found in a follow on PR.

@yanokwa yanokwa self-requested a review May 25, 2022 17:30
@matthew-white matthew-white changed the base branch from master to release May 27, 2022 05:24
@matthew-white matthew-white merged commit 0a5dc9c into getodk:release May 27, 2022
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