Skip to content
This repository has been archived by the owner on Jul 2, 2020. It is now read-only.

fix: fix error control in fc #153

Merged
merged 10 commits into from
May 5, 2020
Merged

fix: fix error control in fc #153

merged 10 commits into from
May 5, 2020

Conversation

czy88840616
Copy link
Member

@czy88840616 czy88840616 commented May 3, 2020

  • 1、修复中间件几处漏掉 await 调用的问题
  • 2、处理阿里云本地调用异常,本地环境将错误堆栈会传给浏览器,其他环境只返回 500

@gitpod-io
Copy link

gitpod-io bot commented May 3, 2020

@codecov-io
Copy link

codecov-io commented May 3, 2020

Codecov Report

Merging #153 into master will decrease coverage by 0.01%.
The diff coverage is 73.68%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #153      +/-   ##
==========================================
- Coverage   80.30%   80.28%   -0.02%     
==========================================
  Files          85       85              
  Lines        3178     3191      +13     
  Branches      353      357       +4     
==========================================
+ Hits         2552     2562      +10     
- Misses        622      625       +3     
  Partials        4        4              
Impacted Files Coverage Δ
packages/serverless-fc-trigger/src/http.ts 67.10% <0.00%> (-1.82%) ⬇️
packages/serverless-fc-starter/src/runtime.ts 85.71% <75.75%> (+0.63%) ⬆️
packages/faas/src/starter.ts 78.46% <100.00%> (ø)
packages/serverless-spec-builder/src/index.ts 91.30% <100.00%> (+0.39%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c3c745d...60958c9. Read the comment docs.

@Lxxyx Lxxyx self-requested a review May 5, 2020 04:53
@czy88840616 czy88840616 merged commit f7dd007 into master May 5, 2020
@czy88840616 czy88840616 deleted the fix_local_dev branch May 5, 2020 04:57
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants