Skip to content

makefile: bind on 0.0.0.0 for livedoc#6336

Merged
rgrinberg merged 1 commit intoocaml:mainfrom
Alizter:ps/rr/makefile__bind_on_0_0_0_0_for_livedoc
Oct 26, 2022
Merged

makefile: bind on 0.0.0.0 for livedoc#6336
rgrinberg merged 1 commit intoocaml:mainfrom
Alizter:ps/rr/makefile__bind_on_0_0_0_0_for_livedoc

Conversation

@Alizter
Copy link
Copy Markdown
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 <alizter@gmail.com>

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
Copy Markdown
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