Merge branch 'main' into add-more-unit-test

.github/.cSpellWords.txt

@@ -49,4 +49,9 @@ utest
 vect
 Vect
 VECT
+Wconversion
+Werror
+Weverything
+Wextra
+Wpedantic
 Wunused

.github/workflows/ci.yml

@@ -16,10 +16,15 @@ jobs:
       - name: Build
         run: |
           sudo apt-get install -y lcov
+
+          # Build the coverity analysis project as well to check compiler warning.
+          # Coverity analysis project builds coreHTTP source file only. llhttp source
+          # files are not built in this target.
           cmake -S test -B build/ \
           -G "Unix Makefiles" \
           -DCMAKE_BUILD_TYPE=Debug \
           -DUNITTEST=1 \
+          -DCOV_ANALYSIS=1 \
           -DCMAKE_C_FLAGS='--coverage -Wall -Wextra -DNDEBUG'
           make -C build/ all
 

.github/workflows/release.yml

@@ -16,7 +16,7 @@ jobs:
     runs-on: ubuntu-latest
     steps:
       - name: Checkout code
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           ref: ${{ github.event.inputs.commit_id }}
       - name: Configure git identity
@@ -53,7 +53,7 @@ jobs:
       - name: Install ZIP tools
         run: sudo apt-get install zip unzip
       - name: Checkout code
-        uses: actions/checkout@v3
+        uses: actions/checkout@v4
         with:
           ref: ${{ github.event.inputs.commit_id }}
           path: coreHTTP
@@ -90,7 +90,7 @@ jobs:
           ctest -E system --output-on-failure
           cd ..
       - name: Create artifact of ZIP
-        uses: actions/upload-artifact@v2
+        uses: actions/upload-artifact@v4
         with:
           name: coreHTTP-${{ github.event.inputs.version_number }}.zip
           path: zip-check/coreHTTP-${{ github.event.inputs.version_number }}.zip
@@ -123,7 +123,7 @@ jobs:
           draft: false
           prerelease: false
       - name: Download ZIP artifact
-        uses: actions/download-artifact@v2
+        uses: actions/download-artifact@v4
         with:
           name: coreHTTP-${{ github.event.inputs.version_number }}.zip
       - name: Upload Release Asset

README.md

@@ -129,7 +129,7 @@ repositories.
 
 ### Generating Documentation
 
-The Doxygen references were created using Doxygen version 1.9.2. To generate the
+The Doxygen references were created using Doxygen version 1.9.6. To generate the
 Doxygen pages, please run the following command from the root of this
 repository:
 

docs/doxygen/pages.dox

@@ -270,7 +270,10 @@ defined.
 @subpage httpclient_initializerequestheaders_function <br>
 @subpage httpclient_addheader_function <br>
 @subpage httpclient_addrangeheader_function <br>
+@subpage httpclient_sendhttpheaders_function <br>
+@subpage httpclient_sendhttpdata_function <br>
 @subpage httpclient_send_function <br>
+@subpage httpclient_receiveandparsehttpresponse_function <br>
 @subpage httpclient_readheader_function <br>
 @subpage httpclient_strerror_function <br>
 
@@ -286,10 +289,22 @@ defined.
 @snippet core_http_client.h declare_httpclient_addrangeheader
 @copydoc HTTPClient_AddRangeHeader
 
+@page httpclient_sendhttpheaders_function HTTPClient_SendHttpHeaders
+@snippet core_http_client.h declare_httpclient_sendhttpheaders
+@copydoc HTTPClient_SendHttpHeaders
+
+@page httpclient_sendhttpdata_function HTTPClient_SendHttpData
+@snippet core_http_client.h declare_httpclient_sendhttpdata
+@copydoc HTTPClient_SendHttpData
+
 @page httpclient_send_function HTTPClient_Send
 @snippet core_http_client.h declare_httpclient_send
 @copydoc HTTPClient_Send
 
+@page httpclient_receiveandparsehttpresponse_function HTTPClient_ReceiveAndParseHttpResponse
+@snippet core_http_client.h declare_httpclient_receiveandparsehttpresponse
+@copydoc HTTPClient_ReceiveAndParseHttpResponse
+
 @page httpclient_readheader_function HTTPClient_ReadHeader
 @snippet core_http_client.h declare_httpclient_readheader
 @copydoc HTTPClient_ReadHeader

source/core_http_client.c

@@ -545,12 +545,12 @@ static int8_t caseInsensitiveStringCmp( const char * str1,
         /* Subtract offset to go from lowercase to uppercase ASCII character */
         if( ( firstChar >= 'a' ) && ( firstChar <= 'z' ) )
         {
-            firstChar = firstChar - offset;
+            firstChar = ( char ) ( firstChar - offset );
         }
 
         if( ( secondChar >= 'a' ) && ( secondChar <= 'z' ) )
         {
-            secondChar = secondChar - offset;
+            secondChar = ( char ) ( secondChar - offset );
         }
 
         if( ( firstChar ) != ( secondChar ) )
@@ -1249,6 +1249,7 @@ static uint8_t convertInt32ToAscii( int32_t value,
     uint8_t numOfDigits = 0U;
     uint8_t index = 0U;
     uint8_t isNegative = 0U;
+    int32_t bufferIndex;
     char temp = '\0';
 
     assert( pBuffer != NULL );
@@ -1263,7 +1264,7 @@ static uint8_t convertInt32ToAscii( int32_t value,
         *pBuffer = '-';
 
         /* Convert the value to its absolute representation. */
-        absoluteValue = value * -1;
+        absoluteValue = value * ( -1 );
     }
 
     /* Write the absolute integer value in reverse ASCII representation. */
@@ -1279,11 +1280,12 @@ static uint8_t convertInt32ToAscii( int32_t value,
     for( index = 0U; index < ( numOfDigits / 2U ); index++ )
     {
         temp = pBuffer[ isNegative + index ];
-        pBuffer[ isNegative + index ] = pBuffer[ isNegative + numOfDigits - index - 1U ];
-        pBuffer[ isNegative + numOfDigits - index - 1U ] = temp;
+        bufferIndex = ( int32_t ) isNegative + ( int32_t ) numOfDigits - ( int32_t ) index - 1;
+        pBuffer[ isNegative + index ] = pBuffer[ bufferIndex ];
+        pBuffer[ bufferIndex ] = temp;
     }
 
-    return( isNegative + numOfDigits );
+    return ( uint8_t ) ( isNegative + numOfDigits );
 }
 
 /*-----------------------------------------------------------*/

test/CMakeLists.txt

@@ -44,16 +44,25 @@ if( COV_ANALYSIS )
 
     # Target for Coverity analysis that builds the library.
     add_library( coverity_analysis
-                ${HTTP_SOURCES} )
+                 ${CMAKE_CURRENT_LIST_DIR}/../source/core_http_client.c )
 
     # Build HTTP library target without custom config dependency.
     target_compile_definitions( coverity_analysis PUBLIC HTTP_DO_NOT_USE_CUSTOM_CONFIG=1 )
 
     # HTTP public include path.
     target_include_directories( coverity_analysis PUBLIC ${HTTP_INCLUDE_PUBLIC_DIRS} )
 
-    # Build HTTP library target without logging
-    target_compile_options(coverity_analysis PUBLIC -DNDEBUG )
+    target_compile_options( coverity_analysis PUBLIC
+        # Build HTTP library target without logging
+        -DNDEBUG
+
+        # GCC compiler option
+        -Wall
+        -Wextra
+        -Wpedantic
+        -Wconversion
+        -Werror
+    )
 endif()
 # ===================== Clone needed third-party libraries ======================