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

makefile: bind on 0.0.0.0 for livedoc #6336

Merged

Conversation

Alizter
Copy link
Collaborator

@Alizter Alizter commented Oct 26, 2022

Some users (me) have permission issues with their hostname.

Some users (me) have permission issues with their hostname.

Signed-off-by: Ali Caglayan <[email protected]>

ps-id: 152b4b0a-dece-439e-91f0-76d8788cc8e8
@Alizter Alizter requested a review from rgrinberg October 26, 2022 20:42
@rgrinberg rgrinberg merged commit 8f1fb28 into ocaml:main Oct 26, 2022
@Alizter Alizter mentioned this pull request Oct 27, 2022
jchavarri added a commit to jchavarri/dune that referenced this pull request Oct 27, 2022
* main:
  makefile: bind on 0.0.0.0 for livedoc (ocaml#6336)
  ci: rewrite fmt job (ocaml#6334)
  refactor(ci): cleanup names (ocaml#6335)
  fix: create fake socket rpc file on windows (ocaml#6329)
  refactor(doc): improvements to hacking.rst (ocaml#6324)
  ci: Add documentation job (ocaml#6333)
  Fix bug in drop_suffix (ocaml#6321)
  fix: public binaries with absolute build path
  test: public binaries in a cram test
  chore: update dune-project to 3.5 (ocaml#6328)
  chore(nix): add `devShells.slim` (ocaml#6327)
  test: remove obsolete bisect tests (ocaml#6318)
  refactor: clarify matching for path comparison and add a docstring (ocaml#6322)
@emillon
Copy link
Collaborator

emillon commented Nov 9, 2022

do we actually need to bind to something more than localhost? it seems that removing the --host argument is both simpler and more secure (not that it matters too much)

@Alizter Alizter deleted the ps/rr/makefile__bind_on_0_0_0_0_for_livedoc branch November 9, 2022 15:06
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.

3 participants