Skip to content

Tags: Tatsuya-2/amazon-kinesis-video-streams-webrtc-sdk-c

Tags

v1.9.1

Toggle v1.9.1's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Remove pIotCoreCertificateId usage from samples. (awslabs#1882)

* Modify compilation errors,when IOT_CORE_ENABLE_CREDENTIALS is enabled

* Use llu

* Remove CertificaqteId usgae in sample

* Remove cert id usage

---------

Co-authored-by: caihy0414 <[email protected]>

v1.9.0

Toggle v1.9.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
testing release with media storage support (awslabs#1783)

* Log ice candidate protocol appropriately (awslabs#1230)

* Log ice candidate protocol appropriately

* Add develop to travis CI

* Address comments

* Realign branches during transition (awslabs#1257)

* create data channel sample (awslabs#1203)

* create data channel sample

* moved variables to Samples.h, encapsulated with ENABLE_DATA_CHANNEL directive

* removed unused variables, moved variable declarations to the top of a block

* create data channel sample

* moved variables to Samples.h, encapsulated with ENABLE_DATA_CHANNEL directive

* Cancel the thread once mediaThreadStarted flag is set to false (awslabs#1227)

* Initial stale PR template (awslabs#1226)

* Update stale issue template

* Testing every minute

* Fix stale issue PR template (awslabs#1234)

No code changed. Just a template change. So merging

* Add bug and question label

* Switch stale PR action to daily cadence

* [FIX] When protocol in DCEP header of data channel open command is not empty, there is a check preventing the data channel to be opened. Now protocol length is correctly handled to avoid check failing. (awslabs#1228)

Co-authored-by: Niyati Maheshwari <[email protected]>
Co-authored-by: Divya Sampath Kumar <[email protected]>
Co-authored-by: ela34 <[email protected]>

* set protocol value following the spec (awslabs#1259)

Co-authored-by: liyufang <[email protected]>

* filter out invalid candidate in sdp (awslabs#1260)

Co-authored-by: liyufang <[email protected]>

* Added targets for cmake clean (awslabs#1253)

* added targets for cmake clean

* added new line, updated readme

* added missing new line

* updated readme to include folder details

* updated readme to include folder details

* viewer sample: wrap datachannel callback with `ENABLE_DATA_CHANNEL` (awslabs#1261)

* create data channel sample (awslabs#1203)

* create data channel sample

* moved variables to Samples.h, encapsulated with ENABLE_DATA_CHANNEL directive

* removed unused variables, moved variable declarations to the top of a block

* create data channel sample

* moved variables to Samples.h, encapsulated with ENABLE_DATA_CHANNEL directive

* Cancel the thread once mediaThreadStarted flag is set to false (awslabs#1227)

* Initial stale PR template (awslabs#1226)

* Update stale issue template

* Testing every minute

* Fix stale issue PR template (awslabs#1234)

No code changed. Just a template change. So merging

* Add bug and question label

* Switch stale PR action to daily cadence

* [FIX] When protocol in DCEP header of data channel open command is not empty, there is a check preventing the data channel to be opened. Now protocol length is correctly handled to avoid check failing. (awslabs#1228)

* viewer sample: wrap datachannel callback with `ENABLE_DATA_CHANNEL`

Signed-off-by: [email protected] <[email protected]>

Co-authored-by: Niyati Maheshwari <[email protected]>
Co-authored-by: Divya Sampath Kumar <[email protected]>
Co-authored-by: ela34 <[email protected]>

* libwebsockets: add patch to fix pipe fd leak issue (awslabs#1264)

Signed-off-by: Alex.Li <[email protected]>

* Fix scripts/pare_status.py for non-status #defines (awslabs#1268) (awslabs#1274)

Co-authored-by: Kevin Allen <[email protected]>

* fix a few bugs

* Revert "fix a few bugs"

This reverts commit 0939024.

* fix some bugs (awslabs#1277)

* Fix bugs (awslabs#1279)

* fix edge case try to unlock a mutex which is not locked in teh acse the sample config was NULL (awslabs#1286)

* cmake: allow user to specify OPEN_SRC_INSTALL_PREFIX (awslabs#1293)

Signed-off-by: Alex.Li <[email protected]>

* Fix inconsistent log priority (awslabs#1309)

Patch reduces priority of a few info level log messages to verbose
level to be consistent with other functions in file and rest of
code base.

* Modify protocol loggin design in develop (awslabs#1311)

* typo: typo fix (awslabs#1317)

Signed-off-by: Alex.Li <[email protected]>

* Signaling state machine rework (awslabs#1323)

* replaced recursive calls to stepSignalingStateMachine with loops

* removed recursive calls to stepSignalingStateMachine

* replaced stepSignalingstatemachine with signalingStateMachineIterator

* removed stepUntil, continueOnReady; removed status from iterator signature; set signalingclient version; added a lock in refreshIceConfiguration

* changed the declaration for i from int to UINT32

* added signaling version; removed unnecessary comment

* removed redeclaration; changed value in an existing macro

* Fix issue with API call failures being treated as success (awslabs#1328)

* return proper error, do not reset call result value

* for non retriable failures, set the terminal exit status for state in… (awslabs#1320)

* for non retriable failures, set the terminal exit status for state in state machine

* address comments

* adjust tests set retry max to 1

* Update LwsApiCalls.c

trigger travis ci

* Incorporating PIC state machine level retry changes into webrtc signaling state machine (awslabs#1326)

* Incorporating PIC state machine level retry changes into webrtc signaling state machine

* Signaling state machine rework (awslabs#1323)

* replaced recursive calls to stepSignalingStateMachine with loops

* removed recursive calls to stepSignalingStateMachine

* replaced stepSignalingstatemachine with signalingStateMachineIterator

* removed stepUntil, continueOnReady; removed status from iterator signature; set signalingclient version; added a lock in refreshIceConfiguration

* changed the declaration for i from int to UINT32

* added signaling version; removed unnecessary comment

* removed redeclaration; changed value in an existing macro

* Fix issue with API call failures being treated as success (awslabs#1328)

* return proper error, do not reset call result value

* for non retriable failures, set the terminal exit status for state in… (awslabs#1320)

* for non retriable failures, set the terminal exit status for state in state machine

* address comments

* adjust tests set retry max to 1

* Update LwsApiCalls.c

trigger travis ci

* Add retry strategy to client info to avoiud changing create signaling channel API signature

* Incorporating PIC state machine level retry changes into webrtc signaling state machine

* Add retry strategy to client info to avoiud changing create signaling channel API signature

* Adding more debug logs in the code

* fix merge conflicts

* PR feedback

Co-authored-by: Niyati Maheshwari <[email protected]>
Co-authored-by: Hassan Sahibzada <[email protected]>

* Revert " Incorporating PIC state machine level retry changes into webrtc signaling state machine (awslabs#1326)" (awslabs#1339)

This reverts commit 06bffce.

* Incorporating PIC state machine level retry changes into webrtc signaling state machine (awslabs#1341)

* Incorporating PIC state machine level retry changes into webrtc signaling state machine

* Add a n optional check for free retry strategy

* Remove unused goto label

* add missing sleep in get token state machine execute API

* update log line

* Update Producer hash

* Adding a high level retry strategy while creating signaling client

* Add Retry count retrieval in hook (awslabs#1335)

* Add Retry count retrieval in hook

* Fix compile issue on travis

* Pull in latest changes in retry structures

* Rebasing off develop with successful travis run for retry

* Log git hash (awslabs#1345)

* Log git hash

* Update CMakeLists.txt

white space change to trigger travis

Co-authored-by: Hassan Sahibzada <[email protected]>

* Fix retry count to indicate count only after a retry has occured (awslabs#1348)

Fix retry count to indicate count only after a retry has occurred

* add support for automatic clock skew detection and correction (awslabs#1344)

* add support for automatic clock skew detection and correction

* auto fix clock skew and add tests

* enable test to check clock recovery

* remove stuff left over from debugging, reset logging default to WARN

* remove unused variable

* get rid of verbose logging in unit tests causes static build to fail due to excessive log output

* simplify code get rid of extra call to length because the lws_hdr_copy returns the same value

* handle null stream track in case of recvonly (awslabs#1346)

* handle null stream track in case of recvonly

* a test with null stream track for revconly

* Fix a comment

* whitespace change

* State machine preparation is decoupled from the constructor (awslabs#1343)

* Merge master into develop (awslabs#1352)

* Missing the public API switch in the other samples (awslabs#1356)

* Fix Fetch error retry cases (awslabs#1359)

* Fix Fetch error retry cases

* update comment

* added implementation and a test to use remote sdp to construct local sdp (awslabs#1238)

* added implementation and a test to use remote sdp to construct local sdp

* used same strings in offer and test

* updated tests

* Update stale action workflow (awslabs#1381)

* Modify issue triaging management

* test on branch

* Update ga for stale issue to use aws-actions instead of the one publicly available

* Travis to ga (awslabs#1347)

* Test github actions

* Clang format

* Change platform

* Add macos gtest

* Ubuntu latest

* Address sanitizer

* Add ubsan, tsan and old gcc build

* Fix macos build

* Allow mac and ubsan to fail

* Fixing clang format

* Testing allow on failure

* Run github actions only on PR and merge

* Test mbedtls

* Rebase on develop and fix clang failure

* Test if travis builds without ga

* Restore travis

* Test mac build and add sample check

* Fix sample check and revert mac to allow fail

* Travis CI to GHA migration phase 1

* Set clang for mbedtls and test msan

* Split files

* Comment out failing tests

* Add codecov

* Travis CI to GHA migration phase 1

* new Chrome v98 produces extra sdp attributes, up the limit so we do n… (awslabs#1391)

* new Chrome v98 produces extra sdp attributes, up the limit so we do not reject

* Fix test and change count to unit16 to avoid overflow issues

* fix issue reported by codeql

Co-authored-by: Divya Sampath Kumar <[email protected]>

* GitHub actions setup (awslabs#1390)

* add additional tests

* fix syntax error

* fix syntax error

* fix syntax error

* fix syntax error

* fix syntax error

* static build and msan without docker

* fix msan

* use dependencies with msan build

* use gcc 9 for mac

* use gcc 4.2 for mac gcc build

* use gcc 4.9 for mac gcc build

* check gcc version; use macos 10.15

* use clang-7 for ubsan

* arm cross compilation

* missed cloning the repository

* use gcc

* remove compiler from env

* fix install deps

* use clang7, macos10.15, ubuntu18.04

* fix windows build

* fix test instruction

* fix static build

* fix static build

* fix static build

* fix static build

* setup docker for static build

* use alpine for the static build

* use alpine for the static build

* install dependencies

* install dependencies

* fix errors in static build

* install dependencies

* enable ipv6

* use gcc for static build

* fix ubsan

* fix ubsan

* collate all working github actions builds

* fix errors

* have static,msan,windows,doxygen in travis

* comment failing builds;fix ci rules

* fix travis comments, os, remove exports

* update log level, mac build-name

* add CC

* update gcc

* update gcc

* check cc and cxx for mac gcc

* Revert "check cc and cxx for mac gcc"

This reverts commit 2db1e0e.

* remove travis statement

* Fix swapped SSL_free() and SSL_CTX_free() (awslabs#1401)

* Hangup value included in LWS retry strategy (awslabs#1403)

* GitHub actions setup static build (awslabs#1405)

* fix static build

* remove static build from travis

* fix warnings in mac builds; setup iam role for sample-check

* Migrate Doxygen to GitHub Actions (awslabs#1416)

* add doxygen workflow to ga

* install graphviz

* install ttf-freefont

* specify doxyfile

* use enc key

* use enc key

* use marketplace action to deploy

* use marketplace action to deploy

* use marketplace action to deploy

* check current folder

* check current folder

* check current folder

* check current folder

* check current folder

* check current folder

* check current folder

* fix deployment

* fix deployment

* fix deployment

* fix deployment

* specify doxyfile path

* fix syntax

* fix paths in doxyfile

* fix forked branch ci

* add develop branch

* Fix ResourceEndpoint length check (awslabs#1411)

ResourceEndpoint length check mistakenly uses ChannelName max length.

* update cross-compilation builds ga (awslabs#1429)

* Addressing error handling concerns for an edge case (awslabs#1443)

* Addressing error handling concerns for an edge case

* Clang format

* Removed useless status_success check, changed test to look for correct error value

* sample: fix possible timerQueue deadlock when freeSampleStreamingSession (awslabs#1448)

Signed-off-by: [email protected] <[email protected]>

* metrics: allow viewer sample to get ice candidate pair stats (awslabs#1406)

Signed-off-by: Alex.Li <[email protected]>

* account for hexidecimal numbers (awslabs#1465)

* Fix default data channel ordering mode (awslabs#1476)

* Invoke gst_object_unref on allocated elements (awslabs#1477)

* Fix typo (awslabs#1478)

* Fix typo

* Add unit test

* find gcc version for static build

* revert gtest version update and change alpine version

* fix memory leak issue and abort trap in test

* fix video delay 4s (awslabs#1525)

* fix mips compile error (awslabs#1532)

* fix mips compile error

* fix mips compile error

* fix mips compile error

* Fix file caching (awslabs#1575)

* Fix file caching

* Update test to count entries, remove unnecessary memcpy

* UT uses MEMFREE

* Fix deadlock getIceCandidatePairStatsCallback (awslabs#1561)

* sample: fix possible timerQueue deadlock when freeSampleStreamingSession

* minor change (formatting)

* wrtc stream ingestion support

* clean up

* default to disable data channel for wrtc ingestion

* Fix missing SDP m-line problem (awslabs#1458)

* use fake transceiver for missing m-line

* use hashtable to track previously seen transceivers

* fix failing tests with updates to seenTransceivers hashtable

* use hashtables for unknown codec rtpmap and payloadtype

* fix firstCodec value in case of only a single codec in m-line

* rename variables

* clang build

* fix mac-gcc build

* add test

* add test for missingAudioTransceiver

* add comments

* add comments

* remove unused variable to fix mac buils

* wrtc stream ingestion support (awslabs#1605)

* update readme (awslabs#1608)

* update readme

* update description

* remove gst based storage sample (awslabs#1619)

* Fix GHA CI build issues (awslabs#1631)

* separate builds and tests

* fix clang format issues

* update libgtest version to 1.12.1

* fix deprecation in libgtest

* update test suite call

* run tests from build

* remove build from tst instruction

* fix san options

* fix windows build

* fix path in windows build

* update bat file for windows

* build openssl on windows

* add openssl path for windows build

* specify path to openssl root folder

* specify path to openssl root folder

* specify path to openssl root folder

* upgrade cmake version

* fix windows build

* update checkoutv2 to v3

* fix checkout and install pthreads

* remove windows build temporarily

* move to v1-node16 for aws credentials

* add sudo clean

* add sudo to both commanda

* use event triggers

* Fix ice candidate issue with GStreamer sample (awslabs#1629)

* fix gst sample

* add branch to ci

* fix clang

* fix os versions in ci

* remove unused vars to fix mac-clang build

* remove unused vars to fix mac-gcc build

* separate builds and tests

* separate builds and tests

* change test instruction for ci

* change test instruction for ci

* fix failing builds

* add branch

* Fix CI for forked branches (awslabs#1634)

* use only target

* use only target

* remove branch name

* blank space change to trigger CI

* fix WSAPoll failed when set POLLPRI flag (awslabs#1636)

Co-authored-by: weishao <[email protected]>

* Update README.md (awslabs#1637)

Co-authored-by: Hassan Sahibzada <[email protected]>

* add paths for fedora (awslabs#1638)

Co-authored-by: Hassan Sahibzada <[email protected]>

* Add clarifying comment for fallthrough (awslabs#1639)

* Bkneff patch 2 (awslabs#1640)

* Update kvsWebRTCClientMasterGstreamerSample.c

Provide an example gstreamer pipeline for Raspberry Pi hardware encoding.  Tested on Raspberry Pi 3

* Clang

---------

Co-authored-by: bkneff <[email protected]>

* thread cancel memory leak, recreate signaling client & lws_context whenever a significant error has occurred, verbose and debug logging for ice & turn (awslabs#1641)

* recreate signaling client & lws_context whenever a significant error has occurred

* clang-format

* recreate signaling client & lws_context whenever a significant error has occurred

* Clang

* adding local and remote null checks

* More verbose and debug logging for ice & turn

* Thread cancel on the media sender thread leads to memory leaks from writeFrame() not freeing all its heap usage

* Clang again

---------

Co-authored-by: James Delaplane <[email protected]>

* wrong port/address on debug log (awslabs#1642)

Co-authored-by: Alex Zhukov <[email protected]>

* Update openssl version (awslabs#1643)

* Update openssl version and fix unused param typo

* Pull in producer C openssl update

* fix(project): fix issue--1614 (awslabs#1644)

Co-authored-by: niedb <[email protected]>

* update versions for actions (awslabs#1647)

* Seggregaated Audio and Video frame handling, updated Master sample to handle audio and video both (awslabs#1646)

Co-authored-by: Vikram(VS) <[email protected]>

* Mark Datachannel specific code explicitly (awslabs#1666)

Co-authored-by: Dmitry Ilyin <[email protected]>

* Fix turn allocation taken long time when credential is incorrect (awslabs#1667)

* fix turn allocation use long time sometimes

* update some code style

* fix compile error

* Update IceAgent.c

* fix clang format

* fix prflx update interrupted connectivity check

* fix clang format

---------

Co-authored-by: Gordon <[email protected]>

* Update RtcpPacket.h (awslabs#1680)

* Update RtcpPacket.h

* Clang format

* Changed sample frames to have 1 second GOP size, and removed overly verbose log (awslabs#1686)

* Jitter buffer overflows (awslabs#1677)

* Modified jitterbuffer to include checks for timestamp overflow

* All tests passing, some new tests still need to be added

* More tests and accompanying fixes

* Resolve coverity finding of multiplication causing type overflow

* Change function order

* Clang format

* Mac compile errors fixed

* Commenting out long-running test, adding timestamp overflow test

* Fix port printf in log (awslabs#1682)

* Fix port printf in log

* Fix the clang-format

Fix the clang-format use ./scripts/clang-format.sh -d src/

* fix turn permission delay 2s (awslabs#1675)

* fix turn permission delay 2s

* Fix the clang-format

Fix the clang-format use ./scripts/clang-format.sh -d src/

* Instruction to build on MacOS M1 (awslabs#1689)

* Fix build dependencies off and add version requirements in readme (awslabs#1691)

* Unlock mutex in case of error (awslabs#1696)

* Unlock in case of error

* fix typos (awslabs#1697)

Co-authored-by: Ben Mesander <[email protected]>

* IoT CertificateId as Stream Name. (awslabs#1721)

* IoT CertificateId as Stream Name.

* Fix the clang-format use ./scripts/clang-format.sh -d src/

---------

Co-authored-by: Lina Chen <[email protected]>

* If local ICE is invalid, return from sending packet. (awslabs#1718)

* If local ICE is invalid, return from sending packet.

* Fix the clang-format use ./scripts/clang-format.sh -d src/

---------

Co-authored-by: Lina Chen <[email protected]>

* Adds RTSP Source to GStreamer Sample (awslabs#1740)

* Adds rtsp source support, modifies source-type determination

* All use cases tested and debugged.

* Clang formatting.

* Clang formatting.

* Updates credentials configuration to fix sample-check error.

* Sample-check fix attempt awslabs#2

* Sample-check fix attempt 3

* Sample-check fix attempt 4

* Sample-check fix attempt

* ""

* " "

* " "

* " "

* Update README.md

* Updates sample run instructions structure, adds RTSP source instructions

* Update README.md

* Renames sample file to be same as executable, updates CMakeLists to reflect this.

* Update README.md

Co-authored-by: Niyati Maheshwari <[email protected]>

* Update README.md

* Updates usage statement to new sample executable name

---------

Co-authored-by: Niyati Maheshwari <[email protected]>

* replace stun with sctp (awslabs#1743)

* Modify the PR template (awslabs#1745)

* modify the PR template

* Use bullets

* Update raspberry pi hardware encode to current gstreamer (awslabs#1746)

* Update raspberry pi hardware encode to current gstreamer

* - Provide build instructions for 32-bit raspbian on 64-bit hardware

---------

Co-authored-by: Ben Mesander <bmesander>

* Set a default user agent if not provided (awslabs#1739)

* Connection timeout sleep (awslabs#1744)

* Remove thread_detach and looping sleep, add thread_join

* Added support for socketpair() and added it to connectionlistener

* Clang formatting

* Fix test that previously expected TID to be invalid

* remove unused variables

* Fixing incorrect typing on comparison

* clang formatting

* strlen -> STRLEN macro

* Off by 1 error in poll rfds

* clang formatting

* Do Not Send SCTP Packets when Data Channel is Disabled (awslabs#1749)

* no sctp packets when data channel is disabled

* fix clang format

* Fix comment typo regarding default source in Gst sample (awslabs#1750)

* Parse IP address if the hostname conforms to Public ipv4 DNS format (awslabs#1759)

* Parse IP address if the hostname conforms to Public ipv4 DNS format

* Unit test for ipIpAddr function

* Include and use threadpool for signaling channel messages (awslabs#1761)

* Include and use threadpool for signaling channel messages

* Fix clang and linux gcc compile errors

* Fix pointer cast

* Using SignalingClientInfo to pass information about Threadpool

* edit README.md

* Not necessary to change struct version

* Update struct version again, add fall-through

* clang formatting

* Clang formatting, again

* Fix the Windows build in the CI (awslabs#1764)

* modify libwebsockets to work on Windows

* set LWS_WITH_STATIC 1

* resume windows ci

* add rest of the options

* fix syntax in ci.yml

* remove other options

* set cmd vard

* missing space in bat

* add missing = in build_windows.bat

* introduce cmake arguments

* find_package openssl for non-windows builds

* static + shared for rest of the builds

* set static + shared true for shared builds always

* add back ssl and crypto paths

* rename vars in libwebsockets' cmake, rearrange, pthread fix

* static=1 always

* remove ssl and crypto paths

* with_threadpool=1

* correction

* correction 2

* correction 3

* correction 4

* correction 5 static

* install gstreamer on windows

* test mbedtls on windows

* use if(WIN32) with set vars

* 1671

* remove openssl arg from mbedtls build

* fix mbedtls windows later

* Introducing profiling of code sections and APIs (awslabs#1755)

* Adds RTSP Source to GStreamer Sample (awslabs#1740)

* Adds rtsp source support, modifies source-type determination

* All use cases tested and debugged.

* Clang formatting.

* Clang formatting.

* Updates credentials configuration to fix sample-check error.

* Sample-check fix attempt awslabs#2

* Sample-check fix attempt 3

* Sample-check fix attempt 4

* Sample-check fix attempt

* ""

* " "

* " "

* " "

* Update README.md

* Updates sample run instructions structure, adds RTSP source instructions

* Update README.md

* Renames sample file to be same as executable, updates CMakeLists to reflect this.

* Update README.md

Co-authored-by: Niyati Maheshwari <[email protected]>

* Update README.md

* Updates usage statement to new sample executable name

---------

Co-authored-by: Niyati Maheshwari <[email protected]>

* Connection timeout sleep (awslabs#1744)

* Remove thread_detach and looping sleep, add thread_join

* Added support for socketpair() and added it to connectionlistener

* Clang formatting

* Fix test that previously expected TID to be invalid

* remove unused variables

* Fixing incorrect typing on comparison

* clang formatting

* strlen -> STRLEN macro

* Off by 1 error in poll rfds

* clang formatting

* Fix comment typo regarding default source in Gst sample (awslabs#1750)

* Initial commit

* New structures to track signaling latency in diagnostics

* Make peer connection stats accessible in application

* Fix ice agent profling to store in variables, new APIs for getting ice agent metrics

* Profile create peer connection

* Macro fixes

* More macro fix

---------

Co-authored-by: Stefan Kieszkowski <[email protected]>
Co-authored-by: Niyati Maheshwari <[email protected]>
Co-authored-by: jdelapla <[email protected]>

* Fix Windows tests (awslabs#1769)

* enable verbose logs

* fix syntax

* without gtest filter

* env path

* cmd.exe /c

* give permissions to file

* move to C: instead of D:

* don't move anything

* add paths to pthread, ssl, crypto, websockets dlls

* cd missing

* uncomment other builds

* comment msan

* filter out tests

* Refactor issue templates (awslabs#1770)

* Update commit

* fix issues

* update ci to run, always memset SignalingFileCacheEntry to zero to avoid cache corruption for uninitialized members, add arn validation and signaling channel name parsing out of arn, add and update tests

* add media storage enabled flag to cache, fix state machine issues with newly introduced states

* clang format

* remove unused, adjust print

* mark unused param as unused

* support temp creds in test

* if needed allocate channel name inside the create validate channel info to properly free, fix cache to actually update items

* clang format

* adjust test timeout

* free channel info in test

* need mutex around initScptSession because it could be called from two different threads, alerted by address sanitizer.  free every instance of channel info

* false to FALSE

* use global init make sctpIsEnabled an atomic bool since it is used per peer connection and can be modified/read across different threads this is necessary to ensure we will not attempt to send any sctp packets when data channel has not been requested in the offer or the client application has not asked for an offer to be constructed with a data channel

* lock around global init/de-init calls, may move this inside the public APIs themselves so they are thread safe

* fail test if could not init signaling

* remove merge left over items

* resolve merge issues

* remove extra sctp init, add aws sdk to test project

* fix data race

* remove unused vars

* testing adding aws sdks

* working tests for join storage session

* wrap up tests changes

* adjust transitions

* fix compile Werror

* fix

* jss metric 0 if no storage

* build on OSX Clang without needing to set CPATH

* build seamlessly Clang

* get libcurl in CI for aws sdk builds

* more ci tweaks

* skip aws sdk related tests when building with gcc4.4

* clang format from update 17.0.2 version of clang-format

* signal jss wait cvar during termination sequence to avoid data race when free siganling client is called and callback routines are still running we may attempt to broadcast this cvar when it was already deleted

* do not build aws sdk or related tests in ci, fix free order to free lock first and then cvar for jssWaitLock

* install libcurl so aws sdk cpp can be installed

* fix issues

* clang fix and set useMediaStorage variable in tests

* copy use media storage field from client supplied channel info

* fix some tests

* differentiate between offer received and offer sent times, fixes tsan issue no longer racing betwen same value in callback, was previously hidden due to entire if block inside messageReceiveFn null check and test sets that to null so that code path was not exercised in test

* fix clang formatting and compile issue

* print error message when we fail to update media storage configuration

* introduce lock to resolve thread data race

* free mutex

* adjust behavior for join session 1 hr termination.  include 44a93a1 mbedtls ci build failure fix for ubuntu 22.04

* clang format and add libcurl to ci build for aws sdk

* connect call will call new join session so need to increment counts.  address some cr comments

* fix clang format

* cleanup, revert some timeout values

* cleanup, revert some timeout defaults

* add correlationId to answer in sample, other cleanup

* when stepping out of join session connect if signaling is still connected move to get ice config so it will be refreshed before next join session call

* update state transition counts based on new transitions

* address remaining PR comments

* cleanup, update readme, enable feature with extra command line arg in sample

* clang formatting

* add media server flag to gst master sample as well and update readme

* remove unused timeout

* update readme

* move code coverage to its own file so auto comment in pr can work

* tag must match exactly linux-gcc-codecov

* add missing jobs tag

---------

Signed-off-by: Alex.Li <[email protected]>
Signed-off-by: [email protected] <[email protected]>
Co-authored-by: Divya Sampath Kumar <[email protected]>
Co-authored-by: jdelapla <[email protected]>
Co-authored-by: Niyati Maheshwari <[email protected]>
Co-authored-by: ela34 <[email protected]>
Co-authored-by: northteam <[email protected]>
Co-authored-by: liyufang <[email protected]>
Co-authored-by: Alex.D.Scofield <[email protected]>
Co-authored-by: Kevin Allen <[email protected]>
Co-authored-by: Chris Hiszpanski <[email protected]>
Co-authored-by: Anurag Katey <[email protected]>
Co-authored-by: Chris Hiszpanski <[email protected]>
Co-authored-by: Gordon <[email protected]>
Co-authored-by: Thomas Ruf <[email protected]>
Co-authored-by: Jeremy Gunawan <[email protected]>
Co-authored-by: weishao <[email protected]>
Co-authored-by: Jeremy Gunawan <[email protected]>
Co-authored-by: bkneff <[email protected]>
Co-authored-by: Alex Zhukov <[email protected]>
Co-authored-by: helloworld098765 <[email protected]>
Co-authored-by: niedb <[email protected]>
Co-authored-by: Vikram Shitole <[email protected]>
Co-authored-by: Vikram(VS) <[email protected]>
Co-authored-by: Dmitry Ilyin <[email protected]>
Co-authored-by: Ben Mesander <[email protected]>
Co-authored-by: Ben Mesander <[email protected]>
Co-authored-by: chenln1124 <[email protected]>
Co-authored-by: Lina Chen <[email protected]>
Co-authored-by: Stefan Kieszkowski <[email protected]>

v1.8.1

Toggle v1.8.1's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Update version (awslabs#1794)

* Update version

* Update commit

v1.8.0

Toggle v1.8.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Update producer dep (awslabs#1776)

* Log ice candidate protocol appropriately (awslabs#1230)

* Log ice candidate protocol appropriately

* Add develop to travis CI

* Address comments

* Realign branches during transition (awslabs#1257)

* create data channel sample (awslabs#1203)

* create data channel sample

* moved variables to Samples.h, encapsulated with ENABLE_DATA_CHANNEL directive

* removed unused variables, moved variable declarations to the top of a block

* create data channel sample

* moved variables to Samples.h, encapsulated with ENABLE_DATA_CHANNEL directive

* Cancel the thread once mediaThreadStarted flag is set to false (awslabs#1227)

* Initial stale PR template (awslabs#1226)

* Update stale issue template

* Testing every minute

* Fix stale issue PR template (awslabs#1234)

No code changed. Just a template change. So merging

* Add bug and question label

* Switch stale PR action to daily cadence

* [FIX] When protocol in DCEP header of data channel open command is not empty, there is a check preventing the data channel to be opened. Now protocol length is correctly handled to avoid check failing. (awslabs#1228)

Co-authored-by: Niyati Maheshwari <[email protected]>
Co-authored-by: Divya Sampath Kumar <[email protected]>
Co-authored-by: ela34 <[email protected]>

* set protocol value following the spec (awslabs#1259)

Co-authored-by: liyufang <[email protected]>

* filter out invalid candidate in sdp (awslabs#1260)

Co-authored-by: liyufang <[email protected]>

* Added targets for cmake clean (awslabs#1253)

* added targets for cmake clean

* added new line, updated readme

* added missing new line

* updated readme to include folder details

* updated readme to include folder details

* viewer sample: wrap datachannel callback with `ENABLE_DATA_CHANNEL` (awslabs#1261)

* create data channel sample (awslabs#1203)

* create data channel sample

* moved variables to Samples.h, encapsulated with ENABLE_DATA_CHANNEL directive

* removed unused variables, moved variable declarations to the top of a block

* create data channel sample

* moved variables to Samples.h, encapsulated with ENABLE_DATA_CHANNEL directive

* Cancel the thread once mediaThreadStarted flag is set to false (awslabs#1227)

* Initial stale PR template (awslabs#1226)

* Update stale issue template

* Testing every minute

* Fix stale issue PR template (awslabs#1234)

No code changed. Just a template change. So merging

* Add bug and question label

* Switch stale PR action to daily cadence

* [FIX] When protocol in DCEP header of data channel open command is not empty, there is a check preventing the data channel to be opened. Now protocol length is correctly handled to avoid check failing. (awslabs#1228)

* viewer sample: wrap datachannel callback with `ENABLE_DATA_CHANNEL`

Signed-off-by: [email protected] <[email protected]>

Co-authored-by: Niyati Maheshwari <[email protected]>
Co-authored-by: Divya Sampath Kumar <[email protected]>
Co-authored-by: ela34 <[email protected]>

* libwebsockets: add patch to fix pipe fd leak issue (awslabs#1264)

Signed-off-by: Alex.Li <[email protected]>

* Fix scripts/pare_status.py for non-status #defines (awslabs#1268) (awslabs#1274)

Co-authored-by: Kevin Allen <[email protected]>

* fix a few bugs

* Revert "fix a few bugs"

This reverts commit 0939024.

* fix some bugs (awslabs#1277)

* Fix bugs (awslabs#1279)

* fix edge case try to unlock a mutex which is not locked in teh acse the sample config was NULL (awslabs#1286)

* cmake: allow user to specify OPEN_SRC_INSTALL_PREFIX (awslabs#1293)

Signed-off-by: Alex.Li <[email protected]>

* Fix inconsistent log priority (awslabs#1309)

Patch reduces priority of a few info level log messages to verbose
level to be consistent with other functions in file and rest of
code base.

* Modify protocol loggin design in develop (awslabs#1311)

* typo: typo fix (awslabs#1317)

Signed-off-by: Alex.Li <[email protected]>

* Signaling state machine rework (awslabs#1323)

* replaced recursive calls to stepSignalingStateMachine with loops

* removed recursive calls to stepSignalingStateMachine

* replaced stepSignalingstatemachine with signalingStateMachineIterator

* removed stepUntil, continueOnReady; removed status from iterator signature; set signalingclient version; added a lock in refreshIceConfiguration

* changed the declaration for i from int to UINT32

* added signaling version; removed unnecessary comment

* removed redeclaration; changed value in an existing macro

* Fix issue with API call failures being treated as success (awslabs#1328)

* return proper error, do not reset call result value

* for non retriable failures, set the terminal exit status for state in… (awslabs#1320)

* for non retriable failures, set the terminal exit status for state in state machine

* address comments

* adjust tests set retry max to 1

* Update LwsApiCalls.c

trigger travis ci

* Incorporating PIC state machine level retry changes into webrtc signaling state machine (awslabs#1326)

* Incorporating PIC state machine level retry changes into webrtc signaling state machine

* Signaling state machine rework (awslabs#1323)

* replaced recursive calls to stepSignalingStateMachine with loops

* removed recursive calls to stepSignalingStateMachine

* replaced stepSignalingstatemachine with signalingStateMachineIterator

* removed stepUntil, continueOnReady; removed status from iterator signature; set signalingclient version; added a lock in refreshIceConfiguration

* changed the declaration for i from int to UINT32

* added signaling version; removed unnecessary comment

* removed redeclaration; changed value in an existing macro

* Fix issue with API call failures being treated as success (awslabs#1328)

* return proper error, do not reset call result value

* for non retriable failures, set the terminal exit status for state in… (awslabs#1320)

* for non retriable failures, set the terminal exit status for state in state machine

* address comments

* adjust tests set retry max to 1

* Update LwsApiCalls.c

trigger travis ci

* Add retry strategy to client info to avoiud changing create signaling channel API signature

* Incorporating PIC state machine level retry changes into webrtc signaling state machine

* Add retry strategy to client info to avoiud changing create signaling channel API signature

* Adding more debug logs in the code

* fix merge conflicts

* PR feedback

Co-authored-by: Niyati Maheshwari <[email protected]>
Co-authored-by: Hassan Sahibzada <[email protected]>

* Revert " Incorporating PIC state machine level retry changes into webrtc signaling state machine (awslabs#1326)" (awslabs#1339)

This reverts commit 06bffce.

* Incorporating PIC state machine level retry changes into webrtc signaling state machine (awslabs#1341)

* Incorporating PIC state machine level retry changes into webrtc signaling state machine

* Add a n optional check for free retry strategy

* Remove unused goto label

* add missing sleep in get token state machine execute API

* update log line

* Update Producer hash

* Adding a high level retry strategy while creating signaling client

* Add Retry count retrieval in hook (awslabs#1335)

* Add Retry count retrieval in hook

* Fix compile issue on travis

* Pull in latest changes in retry structures

* Rebasing off develop with successful travis run for retry

* Log git hash (awslabs#1345)

* Log git hash

* Update CMakeLists.txt

white space change to trigger travis

Co-authored-by: Hassan Sahibzada <[email protected]>

* Fix retry count to indicate count only after a retry has occured (awslabs#1348)

Fix retry count to indicate count only after a retry has occurred

* add support for automatic clock skew detection and correction (awslabs#1344)

* add support for automatic clock skew detection and correction

* auto fix clock skew and add tests

* enable test to check clock recovery

* remove stuff left over from debugging, reset logging default to WARN

* remove unused variable

* get rid of verbose logging in unit tests causes static build to fail due to excessive log output

* simplify code get rid of extra call to length because the lws_hdr_copy returns the same value

* handle null stream track in case of recvonly (awslabs#1346)

* handle null stream track in case of recvonly

* a test with null stream track for revconly

* Fix a comment

* whitespace change

* State machine preparation is decoupled from the constructor (awslabs#1343)

* Merge master into develop (awslabs#1352)

* Missing the public API switch in the other samples (awslabs#1356)

* Fix Fetch error retry cases (awslabs#1359)

* Fix Fetch error retry cases

* update comment

* added implementation and a test to use remote sdp to construct local sdp (awslabs#1238)

* added implementation and a test to use remote sdp to construct local sdp

* used same strings in offer and test

* updated tests

* Update stale action workflow (awslabs#1381)

* Modify issue triaging management

* test on branch

* Update ga for stale issue to use aws-actions instead of the one publicly available

* Travis to ga (awslabs#1347)

* Test github actions

* Clang format

* Change platform

* Add macos gtest

* Ubuntu latest

* Address sanitizer

* Add ubsan, tsan and old gcc build

* Fix macos build

* Allow mac and ubsan to fail

* Fixing clang format

* Testing allow on failure

* Run github actions only on PR and merge

* Test mbedtls

* Rebase on develop and fix clang failure

* Test if travis builds without ga

* Restore travis

* Test mac build and add sample check

* Fix sample check and revert mac to allow fail

* Travis CI to GHA migration phase 1

* Set clang for mbedtls and test msan

* Split files

* Comment out failing tests

* Add codecov

* Travis CI to GHA migration phase 1

* new Chrome v98 produces extra sdp attributes, up the limit so we do n… (awslabs#1391)

* new Chrome v98 produces extra sdp attributes, up the limit so we do not reject

* Fix test and change count to unit16 to avoid overflow issues

* fix issue reported by codeql

Co-authored-by: Divya Sampath Kumar <[email protected]>

* GitHub actions setup (awslabs#1390)

* add additional tests

* fix syntax error

* fix syntax error

* fix syntax error

* fix syntax error

* fix syntax error

* static build and msan without docker

* fix msan

* use dependencies with msan build

* use gcc 9 for mac

* use gcc 4.2 for mac gcc build

* use gcc 4.9 for mac gcc build

* check gcc version; use macos 10.15

* use clang-7 for ubsan

* arm cross compilation

* missed cloning the repository

* use gcc

* remove compiler from env

* fix install deps

* use clang7, macos10.15, ubuntu18.04

* fix windows build

* fix test instruction

* fix static build

* fix static build

* fix static build

* fix static build

* setup docker for static build

* use alpine for the static build

* use alpine for the static build

* install dependencies

* install dependencies

* fix errors in static build

* install dependencies

* enable ipv6

* use gcc for static build

* fix ubsan

* fix ubsan

* collate all working github actions builds

* fix errors

* have static,msan,windows,doxygen in travis

* comment failing builds;fix ci rules

* fix travis comments, os, remove exports

* update log level, mac build-name

* add CC

* update gcc

* update gcc

* check cc and cxx for mac gcc

* Revert "check cc and cxx for mac gcc"

This reverts commit 2db1e0e.

* remove travis statement

* Fix swapped SSL_free() and SSL_CTX_free() (awslabs#1401)

* Hangup value included in LWS retry strategy (awslabs#1403)

* GitHub actions setup static build (awslabs#1405)

* fix static build

* remove static build from travis

* fix warnings in mac builds; setup iam role for sample-check

* Migrate Doxygen to GitHub Actions (awslabs#1416)

* add doxygen workflow to ga

* install graphviz

* install ttf-freefont

* specify doxyfile

* use enc key

* use enc key

* use marketplace action to deploy

* use marketplace action to deploy

* use marketplace action to deploy

* check current folder

* check current folder

* check current folder

* check current folder

* check current folder

* check current folder

* check current folder

* fix deployment

* fix deployment

* fix deployment

* fix deployment

* specify doxyfile path

* fix syntax

* fix paths in doxyfile

* fix forked branch ci

* add develop branch

* Fix ResourceEndpoint length check (awslabs#1411)

ResourceEndpoint length check mistakenly uses ChannelName max length.

* update cross-compilation builds ga (awslabs#1429)

* Addressing error handling concerns for an edge case (awslabs#1443)

* Addressing error handling concerns for an edge case

* Clang format

* Removed useless status_success check, changed test to look for correct error value

* sample: fix possible timerQueue deadlock when freeSampleStreamingSession (awslabs#1448)

Signed-off-by: [email protected] <[email protected]>

* metrics: allow viewer sample to get ice candidate pair stats (awslabs#1406)

Signed-off-by: Alex.Li <[email protected]>

* account for hexidecimal numbers (awslabs#1465)

* Fix default data channel ordering mode (awslabs#1476)

* Invoke gst_object_unref on allocated elements (awslabs#1477)

* Fix typo (awslabs#1478)

* Fix typo

* Add unit test

* find gcc version for static build

* revert gtest version update and change alpine version

* fix memory leak issue and abort trap in test

* fix video delay 4s (awslabs#1525)

* fix mips compile error (awslabs#1532)

* fix mips compile error

* fix mips compile error

* fix mips compile error

* Fix file caching (awslabs#1575)

* Fix file caching

* Update test to count entries, remove unnecessary memcpy

* UT uses MEMFREE

* Fix deadlock getIceCandidatePairStatsCallback (awslabs#1561)

* sample: fix possible timerQueue deadlock when freeSampleStreamingSession

* minor change (formatting)

* Fix missing SDP m-line problem (awslabs#1458)

* use fake transceiver for missing m-line

* use hashtable to track previously seen transceivers

* fix failing tests with updates to seenTransceivers hashtable

* use hashtables for unknown codec rtpmap and payloadtype

* fix firstCodec value in case of only a single codec in m-line

* rename variables

* clang build

* fix mac-gcc build

* add test

* add test for missingAudioTransceiver

* add comments

* add comments

* remove unused variable to fix mac buils

* Fix GHA CI build issues (awslabs#1631)

* separate builds and tests

* fix clang format issues

* update libgtest version to 1.12.1

* fix deprecation in libgtest

* update test suite call

* run tests from build

* remove build from tst instruction

* fix san options

* fix windows build

* fix path in windows build

* update bat file for windows

* build openssl on windows

* add openssl path for windows build

* specify path to openssl root folder

* specify path to openssl root folder

* specify path to openssl root folder

* upgrade cmake version

* fix windows build

* update checkoutv2 to v3

* fix checkout and install pthreads

* remove windows build temporarily

* move to v1-node16 for aws credentials

* add sudo clean

* add sudo to both commanda

* use event triggers

* Fix CI for forked branches (awslabs#1634)

* use only target

* use only target

* remove branch name

* blank space change to trigger CI

* fix WSAPoll failed when set POLLPRI flag (awslabs#1636)

Co-authored-by: weishao <[email protected]>

* Update README.md (awslabs#1637)

Co-authored-by: Hassan Sahibzada <[email protected]>

* add paths for fedora (awslabs#1638)

Co-authored-by: Hassan Sahibzada <[email protected]>

* Add clarifying comment for fallthrough (awslabs#1639)

* Bkneff patch 2 (awslabs#1640)

* Update kvsWebRTCClientMasterGstreamerSample.c

Provide an example gstreamer pipeline for Raspberry Pi hardware encoding.  Tested on Raspberry Pi 3

* Clang

---------

Co-authored-by: bkneff <[email protected]>

* thread cancel memory leak, recreate signaling client & lws_context whenever a significant error has occurred, verbose and debug logging for ice & turn (awslabs#1641)

* recreate signaling client & lws_context whenever a significant error has occurred

* clang-format

* recreate signaling client & lws_context whenever a significant error has occurred

* Clang

* adding local and remote null checks

* More verbose and debug logging for ice & turn

* Thread cancel on the media sender thread leads to memory leaks from writeFrame() not freeing all its heap usage

* Clang again

---------

Co-authored-by: James Delaplane <[email protected]>

* wrong port/address on debug log (awslabs#1642)

Co-authored-by: Alex Zhukov <[email protected]>

* Update openssl version (awslabs#1643)

* Update openssl version and fix unused param typo

* Pull in producer C openssl update

* fix(project): fix issue--1614 (awslabs#1644)

Co-authored-by: niedb <[email protected]>

* update versions for actions (awslabs#1647)

* Seggregaated Audio and Video frame handling, updated Master sample to handle audio and video both (awslabs#1646)

Co-authored-by: Vikram(VS) <[email protected]>

* Mark Datachannel specific code explicitly (awslabs#1666)

Co-authored-by: Dmitry Ilyin <[email protected]>

* Fix turn allocation taken long time when credential is incorrect (awslabs#1667)

* fix turn allocation use long time sometimes

* update some code style

* fix compile error

* Update IceAgent.c

* fix clang format

* fix prflx update interrupted connectivity check

* fix clang format

---------

Co-authored-by: Gordon <[email protected]>

* Update RtcpPacket.h (awslabs#1680)

* Update RtcpPacket.h

* Clang format

* Changed sample frames to have 1 second GOP size, and removed overly verbose log (awslabs#1686)

* Jitter buffer overflows (awslabs#1677)

* Modified jitterbuffer to include checks for timestamp overflow

* All tests passing, some new tests still need to be added

* More tests and accompanying fixes

* Resolve coverity finding of multiplication causing type overflow

* Change function order

* Clang format

* Mac compile errors fixed

* Commenting out long-running test, adding timestamp overflow test

* Fix port printf in log (awslabs#1682)

* Fix port printf in log

* Fix the clang-format

Fix the clang-format use ./scripts/clang-format.sh -d src/

* fix turn permission delay 2s (awslabs#1675)

* fix turn permission delay 2s

* Fix the clang-format

Fix the clang-format use ./scripts/clang-format.sh -d src/

* Instruction to build on MacOS M1 (awslabs#1689)

* Fix build dependencies off and add version requirements in readme (awslabs#1691)

* Unlock mutex in case of error (awslabs#1696)

* Unlock in case of error

* fix typos (awslabs#1697)

Co-authored-by: Ben Mesander <[email protected]>

* IoT CertificateId as Stream Name. (awslabs#1721)

* IoT CertificateId as Stream Name.

* Fix the clang-format use ./scripts/clang-format.sh -d src/

---------

Co-authored-by: Lina Chen <[email protected]>

* If local ICE is invalid, return from sending packet. (awslabs#1718)

* If local ICE is invalid, return from sending packet.

* Fix the clang-format use ./scripts/clang-format.sh -d src/

---------

Co-authored-by: Lina Chen <[email protected]>

* Adds RTSP Source to GStreamer Sample (awslabs#1740)

* Adds rtsp source support, modifies source-type determination

* All use cases tested and debugged.

* Clang formatting.

* Clang formatting.

* Updates credentials configuration to fix sample-check error.

* Sample-check fix attempt awslabs#2

* Sample-check fix attempt 3

* Sample-check fix attempt 4

* Sample-check fix attempt

* ""

* " "

* " "

* " "

* Update README.md

* Updates sample run instructions structure, adds RTSP source instructions

* Update README.md

* Renames sample file to be same as executable, updates CMakeLists to reflect this.

* Update README.md

Co-authored-by: Niyati Maheshwari <[email protected]>

* Update README.md

* Updates usage statement to new sample executable name

---------

Co-authored-by: Niyati Maheshwari <[email protected]>

* replace stun with sctp (awslabs#1743)

* Modify the PR template (awslabs#1745)

* modify the PR template

* Use bullets

* Update raspberry pi hardware encode to current gstreamer (awslabs#1746)

* Update raspberry pi hardware encode to current gstreamer

* - Provide build instructions for 32-bit raspbian on 64-bit hardware

---------

Co-authored-by: Ben Mesander <bmesander>

* Set a default user agent if not provided (awslabs#1739)

* Connection timeout sleep (awslabs#1744)

* Remove thread_detach and looping sleep, add thread_join

* Added support for socketpair() and added it to connectionlistener

* Clang formatting

* Fix test that previously expected TID to be invalid

* remove unused variables

* Fixing incorrect typing on comparison

* clang formatting

* strlen -> STRLEN macro

* Off by 1 error in poll rfds

* clang formatting

* Fix comment typo regarding default source in Gst sample (awslabs#1750)

* Parse IP address if the hostname conforms to Public ipv4 DNS format (awslabs#1759)

* Parse IP address if the hostname conforms to Public ipv4 DNS format

* Unit test for ipIpAddr function

* Include and use threadpool for signaling channel messages (awslabs#1761)

* Include and use threadpool for signaling channel messages

* Fix clang and linux gcc compile errors

* Fix pointer cast

* Using SignalingClientInfo to pass information about Threadpool

* edit README.md

* Not necessary to change struct version

* Update struct version again, add fall-through

* clang formatting

* Clang formatting, again

* Fix the Windows build in the CI (awslabs#1764)

* modify libwebsockets to work on Windows

* set LWS_WITH_STATIC 1

* resume windows ci

* add rest of the options

* fix syntax in ci.yml

* remove other options

* set cmd vard

* missing space in bat

* add missing = in build_windows.bat

* introduce cmake arguments

* find_package openssl for non-windows builds

* static + shared for rest of the builds

* set static + shared true for shared builds always

* add back ssl and crypto paths

* rename vars in libwebsockets' cmake, rearrange, pthread fix

* static=1 always

* remove ssl and crypto paths

* with_threadpool=1

* correction

* correction 2

* correction 3

* correction 4

* correction 5 static

* install gstreamer on windows

* test mbedtls on windows

* use if(WIN32) with set vars

* 1671

* remove openssl arg from mbedtls build

* fix mbedtls windows later

* Introducing profiling of code sections and APIs (awslabs#1755)

* Adds RTSP Source to GStreamer Sample (awslabs#1740)

* Adds rtsp source support, modifies source-type determination

* All use cases tested and debugged.

* Clang formatting.

* Clang formatting.

* Updates credentials configuration to fix sample-check error.

* Sample-check fix attempt awslabs#2

* Sample-check fix attempt 3

* Sample-check fix attempt 4

* Sample-check fix attempt

* ""

* " "

* " "

* " "

* Update README.md

* Updates sample run instructions structure, adds RTSP source instructions

* Update README.md

* Renames sample file to be same as executable, updates CMakeLists to reflect this.

* Update README.md

Co-authored-by: Niyati Maheshwari <[email protected]>

* Update README.md

* Updates usage statement to new sample executable name

---------

Co-authored-by: Niyati Maheshwari <[email protected]>

* Connection timeout sleep (awslabs#1744)

* Remove thread_detach and looping sleep, add thread_join

* Added support for socketpair() and added it to connectionlistener

* Clang formatting

* Fix test that previously expected TID to be invalid

* remove unused variables

* Fixing incorrect typing on comparison

* clang formatting

* strlen -> STRLEN macro

* Off by 1 error in poll rfds

* clang formatting

* Fix comment typo regarding default source in Gst sample (awslabs#1750)

* Initial commit

* New structures to track signaling latency in diagnostics

* Make peer connection stats accessible in application

* Fix ice agent profling to store in variables, new APIs for getting ice agent metrics

* Profile create peer connection

* Macro fixes

* More macro fix

---------

Co-authored-by: Stefan Kieszkowski <[email protected]>
Co-authored-by: Niyati Maheshwari <[email protected]>
Co-authored-by: jdelapla <[email protected]>

* Fix Windows tests (awslabs#1769)

* enable verbose logs

* fix syntax

* without gtest filter

* env path

* cmd.exe /c

* give permissions to file

* move to C: instead of D:

* don't move anything

* add paths to pthread, ssl, crypto, websockets dlls

* cd missing

* uncomment other builds

* comment msan

* filter out tests

* Refactor issue templates (awslabs#1770)

* add support for CN region (awslabs#1612) (awslabs#1777)

Signed-off-by: Alex Li <[email protected]>
Co-authored-by: Alex.D.Scofield <[email protected]>

* sample: remove unused pIotCoreThingName (awslabs#1720)

Signed-off-by: Alex.Li <[email protected]>

* Update commit

* Fix refresh permission test by checking more regularly (awslabs#1779)

* Remove some duplicates and fix some discripancies via rebase with the develop branch

* PR description GHA (awslabs#1781)

* Run unit tests which require creds (awslabs#1780)

* Run unit tests which require creds

* TSAN supp to get past this and check if other tests pass

* Reenable tsan failure

* Isolate tsan

* Enable all tests

* Add locks around connectionClosed modification

* Update SignalingApiFunctionalityTest.cpp

* Have a different signaling send timeout for windows and non-windows platform

* Change timer for timeout test to be unrealistic to actually force a timeout

* Enable tests with codecov

* Update SignalingApiFunctionalityTest.cpp

fix broken test.  future commit will include code change so arn doesn't take the place of the channel name so the channel end point cache numbers will also be low

* Update SignalingApiTest.cpp

if using temp creds need to use the temp creds file format

---------

Co-authored-by: Hassan Sahibzada <[email protected]>
Co-authored-by: James Delaplane <[email protected]>

* Update version string

* Release branch update and readme update (awslabs#1785)

---------

Signed-off-by: Alex.Li <[email protected]>
Signed-off-by: [email protected] <[email protected]>
Signed-off-by: Alex Li <[email protected]>
Co-authored-by: jdelapla <[email protected]>
Co-authored-by: Niyati Maheshwari <[email protected]>
Co-authored-by: ela34 <[email protected]>
Co-authored-by: northteam <[email protected]>
Co-authored-by: liyufang <[email protected]>
Co-authored-by: Alex.D.Scofield <[email protected]>
Co-authored-by: Kevin Allen <[email protected]>
Co-authored-by: Hassan Sahibzada <[email protected]>
Co-authored-by: Chris Hiszpanski <[email protected]>
Co-authored-by: Anurag Katey <[email protected]>
Co-authored-by: Chris Hiszpanski <[email protected]>
Co-authored-by: Gordon <[email protected]>
Co-authored-by: Thomas Ruf <[email protected]>
Co-authored-by: Jeremy Gunawan <[email protected]>
Co-authored-by: weishao <[email protected]>
Co-authored-by: Jeremy Gunawan <[email protected]>
Co-authored-by: bkneff <[email protected]>
Co-authored-by: Alex Zhukov <[email protected]>
Co-authored-by: helloworld098765 <[email protected]>
Co-authored-by: niedb <[email protected]>
Co-authored-by: Vikram Shitole <[email protected]>
Co-authored-by: Vikram(VS) <[email protected]>
Co-authored-by: Dmitry Ilyin <[email protected]>
Co-authored-by: Ben Mesander <[email protected]>
Co-authored-by: Ben Mesander <[email protected]>
Co-authored-by: chenln1124 <[email protected]>
Co-authored-by: Lina Chen <[email protected]>
Co-authored-by: Stefan Kieszkowski <[email protected]>

v1.7.3

Toggle v1.7.3's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Release 1 7 3 (awslabs#1473)

* recreate signaling client & lws_context whenever a significant error has occurred

* Clang format

* More verbose and debug logging for ice & turn

* Clang formatting

* travis be gone

* add github-actions

* more clang formatting

* more clang formatting

* address sanitizer

* fix clang-format

* fix client.c clang-format

Co-authored-by: Niyati Maheshwari <[email protected]>

v1.7.2

Toggle v1.7.2's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Hangup value included in LWS retry strategy (awslabs#1403) (awslabs#1404

)

v1.7.1

Toggle v1.7.1's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
new Chrome v98 produces extra sdp attributes, up the limit so we do n… (

awslabs#1391) (awslabs#1393)

* new Chrome v98 produces extra sdp attributes, up the limit so we do not reject

* Fix test and change count to unit16 to avoid overflow issues

* fix issue reported by codeql

Co-authored-by: Divya Sampath Kumar <[email protected]>

Co-authored-by: Hassan Sahibzada <[email protected]>

v1.7.0

Toggle v1.7.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Release 1 7 0 (awslabs#1354)

* Log ice candidate protocol appropriately (awslabs#1230)

* Log ice candidate protocol appropriately

* Add develop to travis CI

* Address comments

* Realign branches during transition (awslabs#1257)

* create data channel sample (awslabs#1203)

* create data channel sample

* moved variables to Samples.h, encapsulated with ENABLE_DATA_CHANNEL directive

* removed unused variables, moved variable declarations to the top of a block

* create data channel sample

* moved variables to Samples.h, encapsulated with ENABLE_DATA_CHANNEL directive

* Cancel the thread once mediaThreadStarted flag is set to false (awslabs#1227)

* Initial stale PR template (awslabs#1226)

* Update stale issue template

* Testing every minute

* Fix stale issue PR template (awslabs#1234)

No code changed. Just a template change. So merging

* Add bug and question label

* Switch stale PR action to daily cadence

* [FIX] When protocol in DCEP header of data channel open command is not empty, there is a check preventing the data channel to be opened. Now protocol length is correctly handled to avoid check failing. (awslabs#1228)

Co-authored-by: Niyati Maheshwari <[email protected]>
Co-authored-by: Divya Sampath Kumar <[email protected]>
Co-authored-by: ela34 <[email protected]>

* set protocol value following the spec (awslabs#1259)

Co-authored-by: liyufang <[email protected]>

* filter out invalid candidate in sdp (awslabs#1260)

Co-authored-by: liyufang <[email protected]>

* Added targets for cmake clean (awslabs#1253)

* added targets for cmake clean

* added new line, updated readme

* added missing new line

* updated readme to include folder details

* updated readme to include folder details

* viewer sample: wrap datachannel callback with `ENABLE_DATA_CHANNEL` (awslabs#1261)

* create data channel sample (awslabs#1203)

* create data channel sample

* moved variables to Samples.h, encapsulated with ENABLE_DATA_CHANNEL directive

* removed unused variables, moved variable declarations to the top of a block

* create data channel sample

* moved variables to Samples.h, encapsulated with ENABLE_DATA_CHANNEL directive

* Cancel the thread once mediaThreadStarted flag is set to false (awslabs#1227)

* Initial stale PR template (awslabs#1226)

* Update stale issue template

* Testing every minute

* Fix stale issue PR template (awslabs#1234)

No code changed. Just a template change. So merging

* Add bug and question label

* Switch stale PR action to daily cadence

* [FIX] When protocol in DCEP header of data channel open command is not empty, there is a check preventing the data channel to be opened. Now protocol length is correctly handled to avoid check failing. (awslabs#1228)

* viewer sample: wrap datachannel callback with `ENABLE_DATA_CHANNEL`

Signed-off-by: [email protected] <[email protected]>

Co-authored-by: Niyati Maheshwari <[email protected]>
Co-authored-by: Divya Sampath Kumar <[email protected]>
Co-authored-by: ela34 <[email protected]>

* libwebsockets: add patch to fix pipe fd leak issue (awslabs#1264)

Signed-off-by: Alex.Li <[email protected]>

* Fix scripts/pare_status.py for non-status #defines (awslabs#1268) (awslabs#1274)

Co-authored-by: Kevin Allen <[email protected]>

* fix a few bugs

* Revert "fix a few bugs"

This reverts commit 0939024.

* fix some bugs (awslabs#1277)

* Fix bugs (awslabs#1279)

* fix edge case try to unlock a mutex which is not locked in teh acse the sample config was NULL (awslabs#1286)

* cmake: allow user to specify OPEN_SRC_INSTALL_PREFIX (awslabs#1293)

Signed-off-by: Alex.Li <[email protected]>

* Fix inconsistent log priority (awslabs#1309)

Patch reduces priority of a few info level log messages to verbose
level to be consistent with other functions in file and rest of
code base.

* Modify protocol loggin design in develop (awslabs#1311)

* typo: typo fix (awslabs#1317)

Signed-off-by: Alex.Li <[email protected]>

* Signaling state machine rework (awslabs#1323)

* replaced recursive calls to stepSignalingStateMachine with loops

* removed recursive calls to stepSignalingStateMachine

* replaced stepSignalingstatemachine with signalingStateMachineIterator

* removed stepUntil, continueOnReady; removed status from iterator signature; set signalingclient version; added a lock in refreshIceConfiguration

* changed the declaration for i from int to UINT32

* added signaling version; removed unnecessary comment

* removed redeclaration; changed value in an existing macro

* Fix issue with API call failures being treated as success (awslabs#1328)

* return proper error, do not reset call result value

* for non retriable failures, set the terminal exit status for state in… (awslabs#1320)

* for non retriable failures, set the terminal exit status for state in state machine

* address comments

* adjust tests set retry max to 1

* Update LwsApiCalls.c

trigger travis ci

* Incorporating PIC state machine level retry changes into webrtc signaling state machine (awslabs#1326)

* Incorporating PIC state machine level retry changes into webrtc signaling state machine

* Signaling state machine rework (awslabs#1323)

* replaced recursive calls to stepSignalingStateMachine with loops

* removed recursive calls to stepSignalingStateMachine

* replaced stepSignalingstatemachine with signalingStateMachineIterator

* removed stepUntil, continueOnReady; removed status from iterator signature; set signalingclient version; added a lock in refreshIceConfiguration

* changed the declaration for i from int to UINT32

* added signaling version; removed unnecessary comment

* removed redeclaration; changed value in an existing macro

* Fix issue with API call failures being treated as success (awslabs#1328)

* return proper error, do not reset call result value

* for non retriable failures, set the terminal exit status for state in… (awslabs#1320)

* for non retriable failures, set the terminal exit status for state in state machine

* address comments

* adjust tests set retry max to 1

* Update LwsApiCalls.c

trigger travis ci

* Add retry strategy to client info to avoiud changing create signaling channel API signature

* Incorporating PIC state machine level retry changes into webrtc signaling state machine

* Add retry strategy to client info to avoiud changing create signaling channel API signature

* Adding more debug logs in the code

* fix merge conflicts

* PR feedback

Co-authored-by: Niyati Maheshwari <[email protected]>
Co-authored-by: Hassan Sahibzada <[email protected]>

* Revert " Incorporating PIC state machine level retry changes into webrtc signaling state machine (awslabs#1326)" (awslabs#1339)

This reverts commit 06bffce.

* Incorporating PIC state machine level retry changes into webrtc signaling state machine (awslabs#1341)

* Incorporating PIC state machine level retry changes into webrtc signaling state machine

* Add a n optional check for free retry strategy

* Remove unused goto label

* add missing sleep in get token state machine execute API

* update log line

* Update Producer hash

* Adding a high level retry strategy while creating signaling client

* Add Retry count retrieval in hook (awslabs#1335)

* Add Retry count retrieval in hook

* Fix compile issue on travis

* Pull in latest changes in retry structures

* Rebasing off develop with successful travis run for retry

* Log git hash (awslabs#1345)

* Log git hash

* Update CMakeLists.txt

white space change to trigger travis

Co-authored-by: Hassan Sahibzada <[email protected]>

* Fix retry count to indicate count only after a retry has occured (awslabs#1348)

Fix retry count to indicate count only after a retry has occurred

* add support for automatic clock skew detection and correction (awslabs#1344)

* add support for automatic clock skew detection and correction

* auto fix clock skew and add tests

* enable test to check clock recovery

* remove stuff left over from debugging, reset logging default to WARN

* remove unused variable

* get rid of verbose logging in unit tests causes static build to fail due to excessive log output

* simplify code get rid of extra call to length because the lws_hdr_copy returns the same value

* handle null stream track in case of recvonly (awslabs#1346)

* handle null stream track in case of recvonly

* a test with null stream track for revconly

* Fix a comment

* whitespace change

* State machine preparation is decoupled from the constructor (awslabs#1343)

* Merge master into develop (awslabs#1352)

* Missing the public API switch in the other samples (awslabs#1356)

* Fix Fetch error retry cases (awslabs#1359)

* Fix Fetch error retry cases

* update comment

Co-authored-by: Divya Sampath Kumar <[email protected]>
Co-authored-by: Niyati Maheshwari <[email protected]>
Co-authored-by: ela34 <[email protected]>
Co-authored-by: northteam <[email protected]>
Co-authored-by: liyufang <[email protected]>
Co-authored-by: Alex.D.Scofield <[email protected]>
Co-authored-by: Kevin Allen <[email protected]>
Co-authored-by: Hassan Sahibzada <[email protected]>
Co-authored-by: Chris Hiszpanski <[email protected]>
Co-authored-by: Anurag Katey <[email protected]>

v1.6.0

Toggle v1.6.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
fix the bug of media thread. (awslabs#1215)

v1.5.0

Toggle v1.5.0's commit message

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Removing pending queue if session creation fails (awslabs#1061)

* Removing pending queue if session creation fails

* Doing a more thorough job in tracking and cleaning