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

fix: body parser ignore #558

Merged
merged 2 commits into from
Jul 27, 2023
Merged

fix: body parser ignore #558

merged 2 commits into from
Jul 27, 2023

Conversation

elrrrrrrr
Copy link
Member

Closes #551 Some versions encounter issues with audit requests using the GET method & gzip

  • 🧶 Modified the NOT_IMPLEMENTED configuration to skip bodyParser parsing by default.
  • 🧶 Added homeService and implemented miscellaneous methods to handle additional routes.

Closes #551 部分版本 audit 请求为 get 且 参数解析异常

  • 🧶 修改 NOT_IMPLEMENTED 配置,默认跳过 bodyParser 解析
  • 🧶 添加 homeService,提供 misc 方法处理额外路由

@elrrrrrrr elrrrrrrr added the enhancement New feature or request label Jul 27, 2023
@elrrrrrrr elrrrrrrr requested a review from fengmk2 July 27, 2023 07:59
@codecov
Copy link

codecov bot commented Jul 27, 2023

Codecov Report

Merging #558 (2ef655c) into master (8556b5f) will increase coverage by 0.02%.
Report is 1 commits behind head on master.
The diff coverage is 100.00%.

@@            Coverage Diff             @@
##           master     #558      +/-   ##
==========================================
+ Coverage   96.97%   97.00%   +0.02%     
==========================================
  Files         174      175       +1     
  Lines       16681    16709      +28     
  Branches     2190     2201      +11     
==========================================
+ Hits        16177    16208      +31     
+ Misses        504      501       -3     
Files Changed Coverage Δ
app/common/constants.ts 100.00% <100.00%> (ø)
app/core/service/HomeService.ts 100.00% <100.00%> (ø)
app/port/controller/HomeController.ts 100.00% <100.00%> (ø)
config/config.default.ts 88.60% <100.00%> (+0.11%) ⬆️

... and 1 file with indirect coverage changes

@fengmk2
Copy link
Member

fengmk2 commented Jul 27, 2023

auto cr 我删除了,没用

@fengmk2 fengmk2 merged commit 81d6455 into master Jul 27, 2023
12 checks passed
@fengmk2 fengmk2 deleted the body-parser-ignore branch July 27, 2023 08:11
fengmk2 pushed a commit that referenced this pull request Jul 27, 2023
[skip ci]

## [3.39.1](v3.39.0...v3.39.1) (2023-07-27)

### Bug Fixes

* body parser ignore ([#558](#558)) ([81d6455](81d6455))
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
None yet
Development

Successfully merging this pull request may close these issues.

不对所有请求都做 bodyParser
2 participants