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 option to listen on localhost instead of any #33

Open
amatus opened this issue Dec 31, 2015 · 5 comments
Open

Add option to listen on localhost instead of any #33

amatus opened this issue Dec 31, 2015 · 5 comments

Comments

@amatus
Copy link

amatus commented Dec 31, 2015

Better yet, add the ability to pass arbitrary options to jetty to enable things like SSL too (#30).

@SevereOverfl0w
Copy link

For me, it listens on localhost, do you mean the other way around?

@amatus
Copy link
Author

amatus commented Jan 28, 2016

Active Internet connections (servers and established)
Proto Recv-Q Send-Q Local Address           Foreign Address         State       PID/Program name
tcp6       0      0 :::8000                 :::*                    LISTEN      12573/java    

@pandeiro
Copy link
Owner

Sorry for not responding earlier. I agree this would be nice. What would the API look like?

@drownbes
Copy link

drownbes commented Nov 2, 2016

Is there any way to bind it to another than localhost address? Is the project dead? Any alternatives to it?

@Chris-Andrews
Copy link
Contributor

Hey @drownbes: the project is not dead. There is not currently a way to bind to an address other than localhost, but this should be possible to add. As mentioned above, we just need an idea of what the API should look like. I'd suggest possibly adding options like:

k httpkit              bool "already exists and provides option to use httpkit instead of jetty"
K httpkit-config  edn "exposed httpkit config options"
J jetty-config      edn "exposed jetty config options"

The httpkit-config and jetty-config could allow for more complicated things like SSL as @amatus describes above. Alternatively we could simply add the option to use another address as -a address STR or -i ip STR. Exposing both the address and config options would probably be fine as well. I can try to tackle this if it would be useful to people, but would just need to know what @pandeiro thinks about the API options.

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

5 participants