Explain why xhost is needed and address security concerns about appflowy-docs

[MikeWallaceDev]

Hi @gotjoshua thanks for bringing that to our attention. I didn't write the Dockerfile, it was provided by a contributor. Honestly, it doesn't work on my system, so I can't really test it out for alternatives.

What do you suggest that we add to the documentation?
In the future, when referring to the documentation, please use the Documentation repo
You can actually clone the documentation repo and submit a PR with your text.
Thanks!

[gotjoshua]

In the future, when referring to the documentation, please use the Documentation repo

isn't this issue in the docs repo?
(confused)

i actually don't understand it enough to know what to write, thats why i asked...

[MikeWallaceDev]

Oh geez, I'm sorry! Hahaha, my bad :-D

Well, I don't anything about it. The whole Dockerfile was a PR, and I didn't even know what xhost was. I looked into it through your links, and didn't see much to concern me. So that's why I asked you. :-)

Hopefully that Docker container won't be needed soon...

[annieappflowy]

Hi @gotjoshua , @PabloCastellano has made improvements based on the feedback. Can you check if your concern is resolved?

[gotjoshua]

Looks good to me!

[annieappflowy]

Thanks for confirming. Closing the issue now