You are viewing a plain text version of this content. The canonical link for it is here.
Posted to notifications@shenyu.apache.org by GitBox <gi...@apache.org> on 2022/07/02 04:43:54 UTC

[GitHub] [incubator-shenyu] lianjunwei opened a new pull request, #3649: [ISSUE 3623][admin] API document enhancement and Optimization: supports the "spring cloud" plug-in and fix bug.

lianjunwei opened a new pull request, #3649:
URL: https://github.com/apache/incubator-shenyu/pull/3649

   <!-- Describe your PR here; eg. Fixes #issueNo -->
   #3623
   
   1.task1[enhancement]:API document enhancemen: support to pull swaager documents from the "spring cloud" proxy service.
   2.task2[Optimization]:Sandbox test, send request to API and receive response logic optimization.
   3.fix bug: When the back-end micro service port number is 80 and is not explicitly filled in, fix the error problem of pulling documents.  For example, 127.0.0.1:80 is filled in as 127.0.0.1 .
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@shenyu.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-shenyu] yu199195 merged pull request #3649: [ISSUE 3623][admin] API document enhancement and Optimization: supports the "spring cloud" plug-in and fix bug.

Posted by GitBox <gi...@apache.org>.
yu199195 merged PR #3649:
URL: https://github.com/apache/incubator-shenyu/pull/3649


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@shenyu.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [incubator-shenyu] codecov-commenter commented on pull request #3649: [ISSUE 3623][admin] API document enhancement and Optimization: supports the "spring cloud" plug-in and fix bug.

Posted by GitBox <gi...@apache.org>.
codecov-commenter commented on PR #3649:
URL: https://github.com/apache/incubator-shenyu/pull/3649#issuecomment-1172835854

   # [Codecov](https://codecov.io/gh/apache/incubator-shenyu/pull/3649?src=pr&el=h1&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) Report
   > Merging [#3649](https://codecov.io/gh/apache/incubator-shenyu/pull/3649?src=pr&el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (eed95ee) into [master](https://codecov.io/gh/apache/incubator-shenyu/commit/8eed0ccc6e0d839fd5cbdc311a2ef66a2071547a?el=desc&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) (8eed0cc) will **decrease** coverage by `0.43%`.
   > The diff coverage is `26.31%`.
   
   > :exclamation: Current head eed95ee differs from pull request most recent head c9cf0c1. Consider uploading reports for the commit c9cf0c1 to get more accurate results
   
   ```diff
   @@             Coverage Diff              @@
   ##             master    #3649      +/-   ##
   ============================================
   - Coverage     62.60%   62.17%   -0.44%     
   + Complexity     5875     5823      -52     
   ============================================
     Files           888      889       +1     
     Lines         24748    24764      +16     
     Branches       2266     2267       +1     
   ============================================
   - Hits          15494    15397      -97     
   - Misses         7812     7920     +108     
   - Partials       1442     1447       +5     
   ```
   
   
   | [Impacted Files](https://codecov.io/gh/apache/incubator-shenyu/pull/3649?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | Coverage Δ | |
   |---|---|---|
   | [...apache/shenyu/admin/model/dto/ProxyGatewayDTO.java](https://codecov.io/gh/apache/incubator-shenyu/pull/3649/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2hlbnl1LWFkbWluL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGVueXUvYWRtaW4vbW9kZWwvZHRvL1Byb3h5R2F0ZXdheURUTy5qYXZh) | `0.00% <0.00%> (ø)` | |
   | [.../shenyu/admin/service/impl/AppAuthServiceImpl.java](https://codecov.io/gh/apache/incubator-shenyu/pull/3649/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2hlbnl1LWFkbWluL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGVueXUvYWRtaW4vc2VydmljZS9pbXBsL0FwcEF1dGhTZXJ2aWNlSW1wbC5qYXZh) | `77.64% <0.00%> (-0.32%)` | :arrow_down: |
   | [...u/admin/service/manager/impl/SwaggerDocParser.java](https://codecov.io/gh/apache/incubator-shenyu/pull/3649/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2hlbnl1LWFkbWluL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGVueXUvYWRtaW4vc2VydmljZS9tYW5hZ2VyL2ltcGwvU3dhZ2dlckRvY1BhcnNlci5qYXZh) | `0.55% <0.00%> (+<0.01%)` | :arrow_up: |
   | [...ister/AbstractShenyuClientRegisterServiceImpl.java](https://codecov.io/gh/apache/incubator-shenyu/pull/3649/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2hlbnl1LWFkbWluL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGVueXUvYWRtaW4vc2VydmljZS9yZWdpc3Rlci9BYnN0cmFjdFNoZW55dUNsaWVudFJlZ2lzdGVyU2VydmljZUltcGwuamF2YQ==) | `3.77% <ø> (ø)` | |
   | [...che/shenyu/admin/controller/SandboxController.java](https://codecov.io/gh/apache/incubator-shenyu/pull/3649/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2hlbnl1LWFkbWluL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGVueXUvYWRtaW4vY29udHJvbGxlci9TYW5kYm94Q29udHJvbGxlci5qYXZh) | `9.25% <7.89%> (+4.91%)` | :arrow_up: |
   | [.../service/manager/impl/LoadServiceDocEntryImpl.java](https://codecov.io/gh/apache/incubator-shenyu/pull/3649/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2hlbnl1LWFkbWluL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGVueXUvYWRtaW4vc2VydmljZS9tYW5hZ2VyL2ltcGwvTG9hZFNlcnZpY2VEb2NFbnRyeUltcGwuamF2YQ==) | `21.73% <33.33%> (+10.03%)` | :arrow_up: |
   | [...va/org/apache/shenyu/common/enums/RpcTypeEnum.java](https://codecov.io/gh/apache/incubator-shenyu/pull/3649/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2hlbnl1LWNvbW1vbi9zcmMvbWFpbi9qYXZhL29yZy9hcGFjaGUvc2hlbnl1L2NvbW1vbi9lbnVtcy9ScGNUeXBlRW51bS5qYXZh) | `91.30% <33.33%> (-4.16%)` | :arrow_down: |
   | [...ache/shenyu/admin/controller/ApiDocController.java](https://codecov.io/gh/apache/incubator-shenyu/pull/3649/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2hlbnl1LWFkbWluL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGVueXUvYWRtaW4vY29udHJvbGxlci9BcGlEb2NDb250cm9sbGVyLmphdmE=) | `11.11% <100.00%> (+8.08%)` | :arrow_up: |
   | [...apache/shenyu/admin/model/query/SelectorQuery.java](https://codecov.io/gh/apache/incubator-shenyu/pull/3649/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2hlbnl1LWFkbWluL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGVueXUvYWRtaW4vbW9kZWwvcXVlcnkvU2VsZWN0b3JRdWVyeS5qYXZh) | `75.75% <100.00%> (+7.75%)` | :arrow_up: |
   | [...henyu/plugin/logging/kafka/LoggingKafkaPlugin.java](https://codecov.io/gh/apache/incubator-shenyu/pull/3649/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation#diff-c2hlbnl1LXBsdWdpbi9zaGVueXUtcGx1Z2luLWxvZ2dpbmcvc2hlbnl1LXBsdWdpbi1sb2dnaW5nLWthZmthL3NyYy9tYWluL2phdmEvb3JnL2FwYWNoZS9zaGVueXUvcGx1Z2luL2xvZ2dpbmcva2Fma2EvTG9nZ2luZ0thZmthUGx1Z2luLmphdmE=) | `22.72% <0.00%> (-72.73%)` | :arrow_down: |
   | ... and [46 more](https://codecov.io/gh/apache/incubator-shenyu/pull/3649/diff?src=pr&el=tree-more&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation) | |
   
   ------
   
   [Continue to review full report at Codecov](https://codecov.io/gh/apache/incubator-shenyu/pull/3649?src=pr&el=continue&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation)
   > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data`
   > Powered by [Codecov](https://codecov.io/gh/apache/incubator-shenyu/pull/3649?src=pr&el=footer&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Last update [8eed0cc...c9cf0c1](https://codecov.io/gh/apache/incubator-shenyu/pull/3649?src=pr&el=lastupdated&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=The+Apache+Software+Foundation).
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@shenyu.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org