[文档][端口] 使用文档中所写的9000端口 #26
Merged
Conversation
This file contains 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
added 2 commits
January 27, 2020 13:14
rexwangcc
reviewed
Jan 27, 2020
| @@ -46,8 +47,8 @@ app.register_blueprint(data, url_prefix=path_prefix) | |||
| #使用flask蓝图功能来注册http-router | |||
|
|
|||
| if __name__ == '__main__': | |||
| port = os.environ.get("FC_SERVER_PORT", "9000") | |||
There was a problem hiding this comment.
不太了解阿里云，不过 FC_SERVER_PORT 似乎是阿里云Deployment中注册的环境变量，类似GCP或AWS中的env_variables ，保险起见建议改为
port = os.environ.get("FC_SERVER_PORT", "5000")
There was a problem hiding this comment.
There was a problem hiding this comment.
@rexwangcc 谢谢comment，已恢复为9000端口
Merged
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
更正了文档的开发指南写的是9000端口可以调试，实际调试中发现index.py因为没有配置端口而使用的是5000.
次pr修复了这个错误，加入9000作为默认启动端口，否则从
FC_SERVER_PORT环境变量中读取。