Test conv (#635)

qicosmos · Nov 7, 2024 · a5bde4b · a5bde4b
1 parent cc9af8c
commit a5bde4b
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 6 deletions.
diff --git a/.github/workflows/linux_llvm_cov.yml b/.github/workflows/linux_llvm_cov.yml
@@ -13,7 +13,7 @@ jobs:
 
     steps:
       - name: Checkout
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
 
       - name: Install Dependencies
         run: |
@@ -47,10 +47,10 @@ jobs:
           echo "Done!"
 
       - name: Upload Coverage Results
-        uses: actions/upload-artifact@v3
+        uses: actions/upload-artifact@v4
         with:
           name: llvm-cov
-          path: ${{ github.workspace }}/.coverage_llvm_cov
+          path: ${{github.workspace}}/build/.coverage_llvm_cov
 
       - name: Create Code Coverage Report
         working-directory: ${{github.workspace}}/build/tests
@@ -62,7 +62,7 @@ jobs:
           echo "\`\`\`" >> tmp.log
 
       - name: Create Comment
-        uses: peter-evans/create-or-update-comment@v2
+        uses: peter-evans/create-or-update-comment@v4
         with:
           issue-number: ${{ github.event.pull_request.number }}
           body-file: '${{github.workspace}}/build/tests/tmp.log'
diff --git a/include/cinatra/http_parser.hpp b/include/cinatra/http_parser.hpp
@@ -47,7 +47,7 @@ class http_parser {
     }
     if (header_len_ < 0) [[unlikely]] {
       CINATRA_LOG_WARNING << "parse http head failed";
-      if (size == CINATRA_MAX_HTTP_HEADER_FIELD_SIZE) {
+      if (num_headers_ == CINATRA_MAX_HTTP_HEADER_FIELD_SIZE) {
         CINATRA_LOG_ERROR << "the field of http head is out of max limit "
                           << CINATRA_MAX_HTTP_HEADER_FIELD_SIZE
                           << ", you can define macro "
@@ -75,7 +75,7 @@ class http_parser {
 
     if (header_len_ < 0) [[unlikely]] {
       CINATRA_LOG_WARNING << "parse http head failed";
-      if (size == CINATRA_MAX_HTTP_HEADER_FIELD_SIZE) {
+      if (num_headers_ == CINATRA_MAX_HTTP_HEADER_FIELD_SIZE) {
         CINATRA_LOG_ERROR << "the field of http head is out of max limit "
                           << CINATRA_MAX_HTTP_HEADER_FIELD_SIZE
                           << ", you can define macro "