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

Added Authentication #1

Merged
merged 3 commits into from
May 18, 2016
Merged

Added Authentication #1

merged 3 commits into from
May 18, 2016

Conversation

chris1howell
Copy link
Contributor

@chris1howell chris1howell commented May 16, 2016

Added basic authentication with default user and password. The ability to change password and save to EEPROM via the Web interface should be added

Added basic web Authentication
@chris1howell
Copy link
Contributor Author

chris1howell commented May 16, 2016

Changed HTTP to HTTPS on port 443. Added fingerprint for emoncms.org server. Client connection is established HTTPS if the fingerprint matches.

Changed Emon request to HTTPS on port 443 after host cert verification
@TrystanLea
Copy link
Owner

Wow, looks great! shall I fork your original repository and put my changes in there so that this maintains the development history? and then we merge these changes at that point?

Added content types for common file types and added logo image to HTML.
@chris1howell
Copy link
Contributor Author

No need to fork. I am just about done integrating your code.

@TrystanLea TrystanLea merged commit ad8d4e5 into TrystanLea:master May 18, 2016
@TrystanLea
Copy link
Owner

Hello Chris, just testing this, Im relatively new to HTTPS having only recently added TLS support with Lets Encrypt, will the certificate fingerprint change when I renew?
I also had to change it to:
const char* fingerprint = "B6:44:19:FF:B8:F2:62:46:60:39:9D:21:C6:FB:F5:6A:F3:D9:1A:79";
to make it work

@TrystanLea
Copy link
Owner

On another topic, do you know why the access point ssid does not change when I edit:
const char* ssid = "emonESP";
const char* password = "emonesp";

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