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

Sketch of an approach for ' Bind sockets to localhost #398 ' #399

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

dirkx
Copy link

@dirkx dirkx commented Jan 3, 2019

Minor tweak to bind the deamon to localhost -and- have the app do the same (on OSX), to hopefully quell issue #398 and revert the security issue introduced in #331 / #338.

… same (on OSX), to hopefully quell issue mooltipass#398 and revert the security issue introduced in mooltipass#331 / mooltipass#338.
@limpkin
Copy link
Collaborator

limpkin commented Jan 3, 2019

thanks @dirkx !
Let's wait for @raoulh to merge @deXol 's current PR for new device support, then check potential merge conflicts.
PR itself needs to be approved by @raoulh and @deXol

@dirkx
Copy link
Author

dirkx commented Jan 3, 2019 via email

@limpkin
Copy link
Collaborator

limpkin commented Jan 3, 2019

a tested sketch? :)

@dirkx
Copy link
Author

dirkx commented Jan 3, 2019 via email

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