This repository was archived by the owner on Dec 20, 2017. It is now read-only.
允许远程访问调试云代码#92
Closed
marrowleaves wants to merge 3 commits intoleancloud:masterfrom
marrowleaves:master
Closed
Conversation
Contributor
|
非常感谢。 |
Author
|
这个PR只是不希望调试服务器域名hardcode为localhost，这样可以同时允许本地调试和远程访问，可以多人同时调测 |
Contributor
|
@marrowleaves 默认打开的话，对于在非安全环境的人可能会有危险，建议使用开关的方式来实现，比如 avoscloud -H xxx -p 4000 的选项来指定要监听的 host 和端口。 |
Author
|
@aisk 这个改动是希望云函数调用时使用的host和打开云函数调试页面的host一致。考虑安全的话目前 |
Contributor
|
@marrowleaves 0.0.0.0 这个地址就会有风险，因为这个时候服务监听的就是外网。如果当前机器是公网 IP 的话任何人都能访问到开发中的服务。局域网的话也有一定风险。因此开发环境没有特殊原因一定不能监听 0.0.0.0。 |
Author
|
你們本來的 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
3 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
No description provided.