Skip to content

Commit

Permalink
Refactor of endpoint and connection management state logic (#77)
Browse files Browse the repository at this point in the history
* address unbounded channel in signal client

* checkpoint

* checkpoint

* integrate api changes into tx5 high level

* temp test fix

* event cleanup

* checkpoint

* checkpoint

* checkpoint

* checkpoint

* checkpoint

* unit test pattern

* modules

* preflight checking

* ban

* drop edge cases and send timeouts

* broadcast

* stats

* metrics

* demo + benchmarks

* cleanup

* broken test

* fix negotiation

* drop test

* can shutdown signal server

* fix sig drop test

* enable android 33 test

* new rust lints

* attempt windows fix

* windows takes longer

* even more windows time

* switch to app_dirs2 and check permissions on android tests

* checkpoint work on tx5 protocol

* working tx5 proto

* integrate new protocol

* checkpoint, need to cleanup conns on errors

* test fix

* fix panics & closed runtimes

* older rust visibility warn

* try again

* test fix

* win test tweak

* Apply suggestions from code review

Co-authored-by: Jost Schulte <jost.schulte@protonmail.com>

* maybe test fix

* upgrade pion

* debug

* debugging

* event tweak

* event timeout

* add event dispatcher unit tests

* impl Debug on Ep3

* code review comment

* code review comment

* code review comment

* Update crates/tx5/src/proto.rs

Co-authored-by: Michael Dougherty <maackle.d@gmail.com>

* remove convenience conversion per code review

* code review comment

* code review comment

* core review comment

* code review comment

* code review comment

* code review comment - simplified AbortableTimedSharedFuture

* simplify event dispatcher

* cleanup

* switch to ubuntu + kvm for android test runners

* debug config and bump ver

* rebuild lock file

---------

Co-authored-by: Jost Schulte <jost.schulte@protonmail.com>
Co-authored-by: Michael Dougherty <maackle.d@gmail.com>
  • Loading branch information
3 people authored Feb 5, 2024
1 parent fe9d2fa commit d5fa909
Show file tree
Hide file tree
Showing 55 changed files with 4,891 additions and 7,500 deletions.
15 changes: 11 additions & 4 deletions .github/workflows/android.yml
Original file line number Diff line number Diff line change
Expand Up @@ -11,19 +11,26 @@ on:
- main
jobs:
test:
runs-on: macos-latest
runs-on: ubuntu-latest
strategy:
fail-fast: false
matrix:
#android-arch: ["arm64-v8a", "x86_64"]
android-arch: ["x86_64"]
android-api-level: [29]
android-api-level: [29, 33]
android-ndk-version: ["26.0.10792818"]
steps:
- name: Checkout
uses: actions/checkout@v3
uses: actions/checkout@v4

- name: Enable KVM
run: |
echo 'KERNEL=="kvm", GROUP="kvm", MODE="0666", OPTIONS+="static_node=kvm"' | sudo tee /etc/udev/rules.d/99-kvm4all.rules
sudo udevadm control --reload-rules
sudo udevadm trigger --name-match=kvm
- name: Go Toolchain
uses: actions/setup-go@v3
uses: actions/setup-go@v5
with:
go-version: '=1.20.0'

Expand Down
7 changes: 4 additions & 3 deletions .github/workflows/static.yml
Original file line number Diff line number Diff line change
Expand Up @@ -13,20 +13,21 @@ jobs:
static-analysis:
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix:
os: [
ubuntu-latest,
]
toolchain: [
stable,
1.71.1
1.75.0
]
steps:
- name: Checkout
uses: actions/checkout@v2
uses: actions/checkout@v4

- name: Go Toolchain
uses: actions/setup-go@v3
uses: actions/setup-go@v5
with:
go-version: '=1.20.0'

Expand Down
6 changes: 4 additions & 2 deletions .github/workflows/test.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,6 +14,7 @@ jobs:
name: Test
runs-on: ${{ matrix.os }}
strategy:
fail-fast: false
matrix:
os: [
ubuntu-latest,
Expand All @@ -25,10 +26,10 @@ jobs:
]
steps:
- name: Checkout
uses: actions/checkout@v2
uses: actions/checkout@v4

- name: Go Toolchain
uses: actions/setup-go@v3
uses: actions/setup-go@v5
with:
go-version: '=1.20.0'

Expand All @@ -45,5 +46,6 @@ jobs:

- name: Cargo Test
env:
RUST_LOG: error
RUST_BACKTRACE: 1
run: cargo test -- --nocapture
Loading

0 comments on commit d5fa909

Please sign in to comment.