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

Enable SSL on webinterface #48

Closed
sanderux opened this issue Jul 13, 2017 · 7 comments
Closed

Enable SSL on webinterface #48

sanderux opened this issue Jul 13, 2017 · 7 comments

Comments

@sanderux
Copy link
Member

@bkueng This service carries sensitive information. I think it should only be provided over SSL. You can generate a free certificate with letsencrypt. An auto-renewal and easy installer is found here: https://certbot.eff.org/all-instructions/

/cc @LorenzMeier

@bkueng
Copy link
Member

bkueng commented Jul 14, 2017

I completely agree and it's something I have had on my list for quite some time. Just a matter of actually doing it

@bkueng
Copy link
Member

bkueng commented Jul 20, 2017

It wasn't as simple as advertised, but I got it to work: https://logs.px4.io/. I'll add automatic redirects to https after some testing phase.

@bkueng bkueng closed this as completed Jul 20, 2017
@sanderux
Copy link
Member Author

What issues did you have? On debian apache it was really as easy as advertised when i tried

@sanderux
Copy link
Member Author

By the way: perhaps it would be good to redirect http to https by default

@bkueng
Copy link
Member

bkueng commented Jul 20, 2017

What issues did you have? On debian apache it was really as easy as advertised when i tried

Well, first I had to open the port for HTTPS on AWS - that one was obvious. Then I had to figure out if possible and how to make it work for multiple DNS names (logs.px4.io & review.px4.io) with a single certificate. But the most troubles I had with is when I tried the certbot nginx method (I'm using it as a reverse proxy), I just got a timeout without further information as to why. I then tried the manual method which worked, and then then nginx method worked as well, but failed to update the config, so I had to do that myself.

By the way: perhaps it would be good to redirect http to https by default

That's what I meant with:

I'll add automatic redirects to https after some testing phase.

@sanderux
Copy link
Member Author

Ah.. ok, thanks, i'll give everything a test too

@sanderux
Copy link
Member Author

Works fine as far as i've tested

Hi there!

Your uploaded log file with description 'DQ9-AVY Maiden flight 2' is available under:
https://review.px4.io/plot_app?log=eb9956ee-438a-4e9b-8b83-f0547e40cdda

Use the following link to delete the log:
https://review.px4.io/edit_entry?action=delete&log=eb9956ee-438a-4e9b-8b83-f0547e40cdda&token=0fe470040d1f163691ec752f712f69f5

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

No branches or pull requests

2 participants