feat: http-proxy 转发时 支持 axios的所有参数 #2565
Closed
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
|
麻烦补充单测。 |
|
哦哦， 好的。 |
|
在根目录 npm i & npm run bootstrap & npm run build 之后，再进子目录 npm run test |
|
npm run build 失败了 |
|
可能要自行处理下命令，框架的研发本身没有对 windows做支持。 |
|
很神奇， 我在ubuntu下执行 npm run build 报的 不过我直接把命令复制出来就没报错了 |
|
我把修改还原了， 在http-proxy执行 npm test也失败了 在上面的grpc倒是全通过了, 是有什么要注意的吗？ |
|
你可以把单测加了在环境跑。 |
huangapple
force-pushed
the
feat_http-proxy
branch
from
0811bbb
to
91bdad3
Compare
|
看这个报错有什么窍门吗？我搜索我新加的 两个案例 get can redirect host ， get no redirect host 都搜索不到。 |
czy88840616
reviewed
Dec 8, 2022
| @@ -1,6 +1,7 @@ | |||
| import { Configuration, Provide, ServerlessTrigger, ServerlessTriggerType } from '@midwayjs/core'; | |||
| import * as faas from '@midwayjs/faas'; | |||
| import * as proxy from '../../../../src'; | |||
| import https from "https"; | |||
| @@ -1,6 +1,7 @@ | |||
| import { Configuration } from '@midwayjs/core'; | |||
| import * as koa from '@midwayjs/koa'; | |||
| import * as proxy from '../../../../src'; | |||
| import https from "https"; | |||
huangapple
force-pushed
the
feat_http-proxy
branch
from
67b536c
to
8d229c6
Compare
|
现在是proxy的报错了。 |
huangapple
force-pushed
the
feat_http-proxy
branch
2 times, most recently
from
1220a1b
to
4f7810a
Compare
|
@czy88840616 辛苦抽空指导下， 单测有点没搞懂。 |
|
尽量本地跑吧，cd packages/http-proxy & npm run test |
huangapple
force-pushed
the
feat_http-proxy
branch
from
4f7810a
to
8414f4b
Compare
|
哪天有空再研究下吧 |
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.
#2564