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 secure authentication for users #92

Open
eifinger opened this issue May 26, 2018 · 4 comments
Open

Add secure authentication for users #92

eifinger opened this issue May 26, 2018 · 4 comments
Labels
enhancement New feature or request hacktoberfest help wanted Extra attention is needed

Comments

@eifinger
Copy link

Do you have any plans regarding authentication?
I plan to use the GPS feature of the app to keep track of my phone when it's outside my home. But for that I will have to expose the server to the internet and right now everybody could create new families etc.

I would like to have an account with user/password (maybe even certificate) required in order to connect to the server and would prefer not to use my proxy server for that.
What are your thoughts on that?

@schollz
Copy link
Owner

schollz commented May 26, 2018

@eifinger Not really. The main use-case here is for LAN homes so that should be pretty good protection.

Its not too hard to add authentication though, but it needs to be added across the ecosystem (in the app, the cli tool and the server). I have an idea of how to do this, but its not really a priority for me personally. If you'd like though, I'd be happy to help guide you how to do it.

@schollz schollz added enhancement New feature or request help wanted Extra attention is needed labels May 26, 2018
@schollz schollz changed the title Authentication Add secure authentication for users May 26, 2018
@eifinger
Copy link
Author

@schollz
Copy link
Owner

schollz commented Jul 20, 2018

I think something like that would work. In fact you can do it very easily with something like Caddy (basicauth / user pw).

If you do that, then I believe all you need to do is add credentials to your endpoints - https://user:pw@cloud.internalpositioning.com/api/....

I haven't tried this yet, so I think their may be an issue. But let me know and I should be able to fix it.

@Wykks
Copy link
Contributor

Wykks commented Jul 8, 2019

Tested, it's not working (with android client, the web ui works of course). Done it with nginx, and it's sending back 401.

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

No branches or pull requests

3 participants