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

Parsing YML needs to be enhanced #56

Open
Jzow opened this issue Jun 27, 2022 · 7 comments
Open

Parsing YML needs to be enhanced #56

Jzow opened this issue Jun 27, 2022 · 7 comments
Assignees
Labels
enhancement New feature or request

Comments

@Jzow
Copy link
Member

Jzow commented Jun 27, 2022

At present, we can complete TCP listening by taking the port, but we need to monitor the context_path for parsing

  1. Add context_path support fields
  2. Modify the read code of the loading configuration file and add the read context_path field information
@Jzow Jzow added the enhancement New feature or request label Jun 27, 2022
@Jzow
Copy link
Member Author

Jzow commented Jun 27, 2022

@Yangandmore 我不确定这样对macro 宏应该没有什么影响吧,会修改 autoconfigure module 的代码

@Yangandmore
Copy link
Contributor

这里的功能实现可以具体描述一下吗?

@Jzow
Copy link
Member Author

Jzow commented Jun 27, 2022

这里的功能实现可以具体描述一下吗?

可以理解我们需要扩展读取yml配置文件的代码,因为现在只读取了port filed,context_path并没有读取,如果开发者希望接口有前缀,这里所以需要实现解析context_path, 比如

server:
  port: 8080
  context_path: tencent

对应的api url: http:{ip}:8080/tencent

@Jzow
Copy link
Member Author

Jzow commented Jun 27, 2022

@summer-os/back-end-team 这个实现不是很复杂 team 哪位member谁可以实现吗

@Yangandmore
Copy link
Contributor

我知道了,这边是需要实现url链接一下。

@Yangandmore
Copy link
Contributor

晚上有空我这边来弄一下

@Jzow
Copy link
Member Author

Jzow commented Jun 28, 2022

#57 fixed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
Status: No status
Development

No branches or pull requests

3 participants