Squashed commit of the following:

commit 1e2f121 Author: sauwming <[email protected]> Date: Thu Feb 13 10:30:58 2025 +0800 Fixed CI Mac failure (#4304) commit 10b4d30 Author: sauwming <[email protected]> Date: Thu Feb 13 09:06:54 2025 +0800 Audio and video stream refactoring (#4300) commit 6cbf0e6 Author: Riza Sulistyo <[email protected]> Date: Wed Feb 12 15:23:37 2025 +0700 Check if ice strans is valid before using it to send (#4301) commit fdd4041 Author: Maciej Lisowski <[email protected]> Date: Tue Feb 11 12:48:17 2025 +0100 Add missing OnTimerParam import in Android Example (#4299) commit 4ded10f Author: sauwming <[email protected]> Date: Fri Feb 7 13:27:00 2025 +0800 Fixed msg_data assertion in pjsua_acc_send_request() API (#4298) commit 65c4bc9 Author: sauwming <[email protected]> Date: Fri Feb 7 07:18:22 2025 +0800 Fix pjsua sample app user agent (#4296) commit c53ace9 Author: sauwming <[email protected]> Date: Fri Feb 7 07:18:09 2025 +0800 Fixed Java make clean error (#4297) commit e6196ad Author: LeonidGoltsblat <[email protected]> Date: Fri Feb 7 02:15:10 2025 +0300 Aligned memory allocation (#4277) * aligned memory allocaion * Fix alt API implementations (PJ_HAS_POOL_ALT_API) * pool test: add testing for bug in pj_pool_allocate_find with big alignment, and refactor to use unit test API * misc fixes on code review * pool_dbg alignment support + incompatible tests disabled for PJ_HAS_POOL_ALT_API --------- Co-authored-by: bennylp <[email protected]> commit 2fff775 Author: sauwming <[email protected]> Date: Thu Feb 6 13:11:01 2025 +0800 Add API to register custom SDP comparison callback (#4286) commit 99b4d1e Author: sauwming <[email protected]> Date: Thu Feb 6 13:10:38 2025 +0800 Fixed issue with SDP version when reoffer is rejected (#4289) commit 0252152 Author: Benny Prijono <[email protected]> Date: Thu Feb 6 11:09:20 2025 +0700 Use cirunner to capture and analyze GitHub action CI crash (#4288) * Windows runner implementation * Set timeout * Remove initial implementation of ci-runner here (it is on separate repo now) * Remove crash handling (-n) in main.c of unit tests * Install cirunner to CI workflows * Adding crash to timestamp test * Fix missing cirunner in one of the job * Reinstall core_pattern on Linux * Removed intentional crash in timestamp_test() * Upload program and core dump on crash * Add crash code in uri_test.c * Removed injected crash in uri_test. Disable stdout/stderr buffering for unit tests * Minor: remove space left out by previous clean up commit 3fcce51 Author: sauwming <[email protected]> Date: Wed Feb 5 11:43:27 2025 +0800 Fixed OpenSSL log error reading cert (#4291) commit cbfbbc4 Author: jimying <[email protected]> Date: Wed Feb 5 11:03:53 2025 +0800 iocp: fix crash, GetQueuedCompletionStatus() write freed WSAOVERLAPPED memory (#4136) commit 205baf0 Author: sauwming <[email protected]> Date: Tue Feb 4 17:04:25 2025 +0800 Fixed warnings in sip auth client (#4287) commit abffe0d Author: sauwming <[email protected]> Date: Tue Feb 4 08:38:55 2025 +0800 Fixed CI test failure (#4284) commit 986fc78 Author: Johannes <[email protected]> Date: Mon Feb 3 08:31:28 2025 +0100 Share an auth session between multiple dialogs/regc (#4262) commit 46111c4 Author: Nanang Izzuddin <[email protected]> Date: Mon Feb 3 11:36:54 2025 +0700 Best effort avoid crash when media transport adapter not using group lock (#4281) commit f986ad8 Author: Benny Prijono <[email protected]> Date: Fri Jan 31 09:45:19 2025 +0700 Add link to coding style documentation (#4280) commit 727ee32 Author: Nanang Izzuddin <[email protected]> Date: Fri Jan 31 09:08:02 2025 +0700 Fix build error when PJ_LOG_MAX_LEVEL is zero (#4279) The `pj_log_get_log_func()` is not defined when PJ_LOG_MAX_LEVEL is set to zero. Thanks to Giorgio Alfarano for the report. commit dae52f6 Author: Perry Ismangil <[email protected]> Date: Thu Jan 30 08:43:54 2025 +0000 Fixing typo (#4274) Acoustic commit 1a4cd67 Author: sauwming <[email protected]> Date: Thu Jan 30 15:21:49 2025 +0800 Modify iOS sample apps dev team ID (#4278) commit dfcfa13 Author: Tarteszeus <[email protected]> Date: Thu Jan 30 02:42:36 2025 +0100 Add queried names to server address record, and add the address record in parameter for on_verify_cb callback (#4256) commit f9e56d8 Author: Jan Tojnar <[email protected]> Date: Wed Jan 29 07:42:18 2025 +0100 Fix duplicate function name in 100rel docs (#4275) commit 960597e Author: Nanang Izzuddin <[email protected]> Date: Wed Jan 29 13:36:34 2025 +0700 Various works on SWIG Java (#4273) * Various works on SWIG Java 1. Fix type mapping (SWIGTYPE_*): a. Map C "void*" & "void**" to Java long (was SWIGTYPE_p_void & SWIGTYPE_p_p_void which are not really usable), this should fix #4242. b. Map pjmedia_aud_dev_index to int. c. Map unsigned char[20] for SslCertInfo.serialNo to Java "short array" This also updates pjsua.i, e.g: tab->space, reorder things. 2. Update swig_java_pjsua2.vcxproj: a. Rename config "Debug" & "Release" to "Debug-Dynamic" & "Release-Dynamic" in , as the project actually builds dynamic libs. Also fix the property sheet dependencies from *-static to *-dynamic. b. Update other settings, e.g: built tool version from 140 to 143. 3. Update symbols.lst: added missing new types, tab->space, reorder alphabetically. * Update ci-win.yml * Add sample code for passing user data using utilTimerSchedule() * Add sample for cancelling timer commit c36ed2c Author: Benny Prijono <[email protected]> Date: Tue Jan 28 16:58:55 2025 +0700 Minor modifications to Android build and samples to match new documentation (#4271) * To streamline the command, also clean swig and pjsua jni output directories when make distclean and realclean is called * Kotlin sample: add account, modify video size and bandwidth, and audio codec priorities to use AMR-WB * Android CLI app: fix armeabi hardcoded arch and also copy stdc++.so commit bab33d6 Author: Noel Morgan <[email protected]> Date: Tue Jan 28 01:51:50 2025 -0600 Added support for updated RFC7866 content-type sub type with XML extension (#4270) commit a89917e Author: sauwming <[email protected]> Date: Fri Jan 24 14:49:59 2025 +0800 OpenSSL: Set ciphersuites only if not using BoringSSL (#4269) commit 377a80c Author: Nanang Izzuddin <[email protected]> Date: Fri Jan 24 13:48:31 2025 +0700 Fix various compile errors & warnings in MSVC2005 (#4268) commit de3f2e1 Author: sauwming <[email protected]> Date: Fri Jan 24 10:20:28 2025 +0800 Set CI vars in GH workflow file (#4263) commit cdb1294 Author: sauwming <[email protected]> Date: Thu Jan 23 11:01:27 2025 +0800 Various fixes for Apple SSL backend (#4257)
pjsip · Feb 13, 2025 · da00a83 · da00a83
1 parent e1064d1
commit da00a83
Show file tree

Hide file tree

Showing 97 changed files with 4,820 additions and 4,217 deletions.
diff --git a/.github/PULL_REQUEST_TEMPLATE/pull_request_template.md b/.github/PULL_REQUEST_TEMPLATE/pull_request_template.md
@@ -23,7 +23,7 @@
 ## Checklist:
 <!--- Go over all the following points, and put an `x` in all the boxes that apply. -->
 <!--- If you're unsure about any of these, don't hesitate to ask. We're here to help! -->
-- [ ] My code follows the code style of this project.
+- [ ] My code follows the **[CODING STYLE of this project](https://docs.pjsip.org/en/latest/get-started/coding-style.html)**.
 - [ ] My change requires a change to the documentation.
 - [ ] I have updated the documentation accordingly.
 - [ ] I have read the **CONTRIBUTING** document.

diff --git a/.github/workflows/ci-linux.yml b/.github/workflows/ci-linux.yml
@@ -6,8 +6,10 @@ on:
   pull_request:
     types: [opened, synchronize, reopened]
 env:
-  CI_ARGS: ${{ vars.CI_UBUNTU_ARGS }}
-  CI_MODE: ${{ vars.CI_MODE }}
+  CI_ARGS: -w 3 --shuffle --stdout-buf 1
+  CI_MODE: --ci-mode
+  # Setting CI_RUNNER to empty should disable cirunner
+  CI_RUNNER: python ${{ github.workspace }}/cirunner/cirunner.py -t 3600 -o ${{ github.workspace }}/artifacts --
   MAKE_FAST: make -j 3
 jobs:
   default-build:
@@ -16,6 +18,10 @@ jobs:
     name: Default / build only
     steps:
     - uses: actions/checkout@v2
+    - name: install cirunner
+      run: |
+        git clone --depth 1 https://github.com/pjsip/cirunner.git
+        cirunner/installlinux.sh
     - name: configure
       run: ./configure
     - name: make
@@ -24,15 +30,25 @@ jobs:
       run: pjlib/bin/pjlib-test-`make infotarget` --config --list | grep SSL
     - name: verify oepnssl is used
       run: pjlib/bin/pjlib-test-`make infotarget` --config --list | grep -E 'PJ_SSL_SOCK_IMP\s+:\s+1'
+    - name: upload artifacts on failure
+      if: ${{ failure() }}
+      uses: actions/upload-artifact@v4
+      with:
+        name: ${{ runner.os }}-${{ runner.arch }}-${{ github.job }}-${{ github.run_id }}
+        path: artifacts
 
   default-full-bundle-1:
     # full bundle: enable all codecs + AEC + DTLS
     runs-on: ubuntu-latest
     name: Default / pjmedia,pjsua
     steps:
     - uses: actions/checkout@v2
+    - name: install cirunner
+      run: |
+        git clone --depth 1 https://github.com/pjsip/cirunner.git
+        cirunner/installlinux.sh
     - name: install dependencies
-      run: sudo apt-get install -y swig sip-tester libopencore-amrnb-dev
+      run: sudo apt-get update && sudo apt-get install -y swig sip-tester libopencore-amrnb-dev
     - name: config site
       run: cd pjlib/include/pj && cp config_site_test.h config_site.h
     - name: configure
@@ -61,12 +77,22 @@ jobs:
       run: make pjmedia-test
     - name: pjsua-test
       run: make pjsua-test
+    - name: upload artifacts on failure
+      if: ${{ failure() }}
+      uses: actions/upload-artifact@v4
+      with:
+        name: ${{ runner.os }}-${{ runner.arch }}-${{ github.job }}-${{ github.run_id }}
+        path: artifacts
 
   default-full-bundle-2:
     runs-on: ubuntu-latest
     name: Default / pjlib,util,pjnath
     steps:
     - uses: actions/checkout@v2
+    - name: install cirunner
+      run: |
+        git clone --depth 1 https://github.com/pjsip/cirunner.git
+        cirunner/installlinux.sh
     - name: install dependencies
       run: sudo apt-get install -y libopencore-amrnb-dev
     - name: config site
@@ -81,12 +107,22 @@ jobs:
       run: make pjlib-util-test
     - name: pjnath-test
       run: make pjnath-test
+    - name: upload artifacts on failure
+      if: ${{ failure() }}
+      uses: actions/upload-artifact@v4
+      with:
+        name: ${{ runner.os }}-${{ runner.arch }}-${{ github.job }}-${{ github.run_id }}
+        path: artifacts
 
   default-full-bundle-3:
     runs-on: ubuntu-latest
     name: Default / pjsip
     steps:
     - uses: actions/checkout@v2
+    - name: install cirunner
+      run: |
+        git clone --depth 1 https://github.com/pjsip/cirunner.git
+        cirunner/installlinux.sh
     - name: install dependencies
       run: sudo apt-get install -y libopencore-amrnb-dev
     - name: config site
@@ -97,12 +133,22 @@ jobs:
       run: $MAKE_FAST
     - name: pjsip-test
       run: make pjsip-test
+    - name: upload artifacts on failure
+      if: ${{ failure() }}
+      uses: actions/upload-artifact@v4
+      with:
+        name: ${{ runner.os }}-${{ runner.arch }}-${{ github.job }}-${{ github.run_id }}
+        path: artifacts
 
   no-tls:
     runs-on: ubuntu-latest
     name: No SSL / pjlib,pjsip
     steps:
     - uses: actions/checkout@v2
+    - name: install cirunner
+      run: |
+        git clone --depth 1 https://github.com/pjsip/cirunner.git
+        cirunner/installlinux.sh
     - name: install dependencies
       run: sudo apt-get install -y swig
     - name: configure
@@ -115,6 +161,12 @@ jobs:
       run: make pjlib-test
     - name: pjsip-test
       run: make pjsip-test
+    - name: upload artifacts on failure
+      if: ${{ failure() }}
+      uses: actions/upload-artifact@v4
+      with:
+        name: ${{ runner.os }}-${{ runner.arch }}-${{ github.job }}-${{ github.run_id }}
+        path: artifacts
 
   # build-ubuntu-openssl
   # TLS: with OpenSSL (same as build-ubuntu-default)
@@ -124,6 +176,10 @@ jobs:
     name: GnuTLS / pjlib,pjnath,pjsip
     steps:
     - uses: actions/checkout@v2
+    - name: install cirunner
+      run: |
+        git clone --depth 1 https://github.com/pjsip/cirunner.git
+        cirunner/installlinux.sh
     - name: install dependencies
       run: sudo apt-get update && sudo apt-get install -y --fix-missing swig libgnutls28-dev
     - name: configure
@@ -142,14 +198,24 @@ jobs:
       run: make pjnath-test
     - name: pjsip-test
       run: make pjsip-test
+    - name: upload artifacts on failure
+      if: ${{ failure() }}
+      uses: actions/upload-artifact@v4
+      with:
+        name: ${{ runner.os }}-${{ runner.arch }}-${{ github.job }}-${{ github.run_id }}
+        path: artifacts
 
   vid-openh264-1:
     runs-on: ubuntu-latest
     name: OpenH264+VPX / pjmedia,pjsua
     steps:
     - uses: actions/checkout@v2
+    - name: install cirunner
+      run: |
+        git clone --depth 1 https://github.com/pjsip/cirunner.git
+        cirunner/installlinux.sh
     - name: install dependencies
-      run: sudo apt-get install -y swig nasm sip-tester libvpx-dev libopencore-amrnb-dev libsdl2-dev
+      run: sudo apt-get update && sudo apt-get install -y swig nasm sip-tester libvpx-dev libopencore-amrnb-dev libsdl2-dev
     - name: get openh264
       run: git clone --depth 1 --single-branch --branch openh264v2.1.0 https://github.com/cisco/openh264.git
     - name: build openh264
@@ -185,18 +251,31 @@ jobs:
     # SDL error: no available vid dev
     #- name: ensure SDL is installed
     #  run: cat pjsua-caps | grep -E '\[SDL\]\[render\]'
+      # one of the step above has changed core_pattern, restore it
+    - name: reinstall cirunner
+      run: cirunner/installlinux.sh
     - name: pjmedia-test
       run: make pjmedia-test
     - name: pjsua-test
       run: make pjsua-test
+    - name: upload artifacts on failure
+      if: ${{ failure() }}
+      uses: actions/upload-artifact@v4
+      with:
+        name: ${{ runner.os }}-${{ runner.arch }}-${{ github.job }}-${{ github.run_id }}
+        path: artifacts
 
   vid-openh264-2:
     runs-on: ubuntu-latest
     name: OpenH264+VPX / pjlib,util,pjnath
     steps:
     - uses: actions/checkout@v2
+    - name: install cirunner
+      run: |
+        git clone --depth 1 https://github.com/pjsip/cirunner.git
+        cirunner/installlinux.sh
     - name: install dependencies
-      run: sudo apt-get install -y nasm libvpx-dev libopencore-amrnb-dev libsdl2-dev
+      run: sudo apt-get update && sudo apt-get install -y nasm libvpx-dev libopencore-amrnb-dev libsdl2-dev
     - name: get openh264
       run: git clone --depth 1 --single-branch --branch openh264v2.1.0 https://github.com/cisco/openh264.git
     - name: build openh264
@@ -207,20 +286,33 @@ jobs:
       run: CFLAGS="-g" LDFLAGS="-rdynamic" ./configure
     - name: make
       run: $MAKE_FAST
+      # one of the step above has changed core_pattern, restore it
+    - name: reinstall cirunner
+      run: cirunner/installlinux.sh
     - name: pjlib-test
       run: make pjlib-test
     - name: pjlib-util-test
       run: make pjlib-util-test
     - name: pjnath-test
       run: make pjnath-test
+    - name: upload artifacts on failure
+      if: ${{ failure() }}
+      uses: actions/upload-artifact@v4
+      with:
+        name: ${{ runner.os }}-${{ runner.arch }}-${{ github.job }}-${{ github.run_id }}
+        path: artifacts
 
   vid-openh264-3:
     runs-on: ubuntu-latest
     name: OpenH264+VPX / pjsip
     steps:
     - uses: actions/checkout@v2
+    - name: install cirunner
+      run: |
+        git clone --depth 1 https://github.com/pjsip/cirunner.git
+        cirunner/installlinux.sh
     - name: install dependencies
-      run: sudo apt-get install -y nasm libvpx-dev libopencore-amrnb-dev libsdl2-dev
+      run: sudo apt-get update && sudo apt-get install -y nasm libvpx-dev libopencore-amrnb-dev libsdl2-dev
     - name: get openh264
       run: git clone --depth 1 --single-branch --branch openh264v2.1.0 https://github.com/cisco/openh264.git
     - name: build openh264
@@ -231,16 +323,29 @@ jobs:
       run: CFLAGS="-g" LDFLAGS="-rdynamic" ./configure
     - name: make
       run: $MAKE_FAST
+      # one of the step above has changed core_pattern, restore it
+    - name: reinstall cirunner
+      run: cirunner/installlinux.sh
     - name: pjsip-test
       run: make pjsip-test
+    - name: upload artifacts on failure
+      if: ${{ failure() }}
+      uses: actions/upload-artifact@v4
+      with:
+        name: ${{ runner.os }}-${{ runner.arch }}-${{ github.job }}-${{ github.run_id }}
+        path: artifacts
 
   vid-ffmpeg:
     runs-on: ubuntu-latest
     name: FFMPEG+x264 / pjmedia
     steps:
     - uses: actions/checkout@v2
+    - name: install cirunner
+      run: |
+        git clone --depth 1 https://github.com/pjsip/cirunner.git
+        cirunner/installlinux.sh
     - name: install dependencies
-      run: sudo apt-get install -y swig nasm libx264-dev libvpx-dev libsdl2-dev
+      run: sudo apt-get update && sudo apt-get install -y swig nasm libx264-dev libvpx-dev libsdl2-dev
     - name: get ffmpeg
       run: git clone --depth 1 --single-branch --branch release/4.2 https://github.com/FFmpeg/FFmpeg.git
     - name: configure ffmpeg
@@ -273,5 +378,14 @@ jobs:
     # SDL error: no available vid dev
     #- name: ensure SDL is installed
     #  run: cat pjsua-caps | grep -E '\[SDL\]\[render\]'
+      # one of the step above has changed core_pattern, restore it
+    - name: reinstall cirunner
+      run: cirunner/installlinux.sh
     - name: pjmedia-test
       run: export LD_LIBRARY_PATH=/usr/local/lib:$LD_LIBRARY_PATH && make pjmedia-test
+    - name: upload artifacts on failure
+      if: ${{ failure() }}
+      uses: actions/upload-artifact@v4
+      with:
+        name: ${{ runner.os }}-${{ runner.arch }}-${{ github.job }}-${{ github.run_id }}
+        path: artifacts