Ecosyste.ms: Timeline

Browse the timeline of events for every public repo on GitHub. Data updated hourly from GH Archive.

algesten/str0m

thomaseizinger created a comment on a pull request on algesten/str0m
> @thomaseizinger you might need to add > > "feature["sha1"]" > > on your side after this change. That is a semver-breaking change then. Is the plan to make the corresponding version bump ...

View on GitHub

xnorpx created a comment on a pull request on algesten/str0m
@thomaseizinger you might need to add "feature["sha1"]" on your side after this change.

View on GitHub

xnorpx created a comment on a pull request on algesten/str0m
@algesten ok I tested it in our repo and end2end and seem to work fine. So ready for merge unless more comments.

View on GitHub

xnorpx created a review comment on a pull request on algesten/str0m
gah

View on GitHub

xnorpx created a review on a pull request on algesten/str0m

View on GitHub

algesten created a review comment on a pull request on algesten/str0m
Not disabled by default anymore :)

View on GitHub

algesten created a review on a pull request on algesten/str0m

View on GitHub

xnorpx created a comment on a pull request on algesten/str0m
> I think we should separate the idea of win crypto vs openssl and this. The `sha1` crate is effectively an optimization regardless of which "base" crypto provider you select. > > Ultimately the...

View on GitHub

xnorpx created a comment on a pull request on algesten/str0m
> I think we should separate the idea of win crypto vs openssl and this. The `sha1` crate is effectively an optimization regardless of which "base" crypto provider you select. > > Ultimately the...

View on GitHub

algesten created a comment on a pull request on algesten/str0m
I think we should separate the idea of win crypto vs openssl and this. The `sha1` crate is effectively an optimization regardless of which "base" crypto provider you select. Ultimately the featu...

View on GitHub

xnorpx created a comment on a pull request on algesten/str0m
I will run this change through our CI and deployment and also figure out github

View on GitHub

xnorpx opened a pull request on algesten/str0m
Make sha1 crate optional and a feature
For security reasons rust crypto might need to be disabled and one would want use pure OpenSSL. This change adds a feature flag for sha1 crate. - Allows users to not use sha1 and only use Open...
algesten deleted a branch algesten/str0m

dedupe-remote-ice-candidates

algesten pushed 1 commit to main algesten/str0m
  • Dedupe remote ICE candidates The changes in `5c4c9d5` caused problems because we'd end up creating new identical pa... 41ce6cf

View on GitHub

algesten closed a pull request on algesten/str0m
Dedupe remote ICE candidates
The changes in `5c4c9d5` caused problems because we'd end up creating new identical pairs. For example, if an existing pair, `0-0`, exists as local: 192.168.1.33:5445, remote: 192.168.1.33:63461. T...
algesten pushed 1 commit to dedupe-remote-ice-candidates algesten/str0m

View on GitHub

algesten created a review comment on a pull request on algesten/str0m
Maybe. But let's revise that in another PR.

View on GitHub

algesten created a review on a pull request on algesten/str0m

View on GitHub

algesten created a review on a pull request on algesten/str0m

View on GitHub

k0nserv created a review comment on a pull request on algesten/str0m
Maybe this business should be encapsulated in some method `Candidate::equivalent`

View on GitHub

k0nserv created a review on a pull request on algesten/str0m

View on GitHub

lolgesten created a review on a pull request on algesten/str0m

View on GitHub

k0nserv opened a pull request on algesten/str0m
Dedupe remote ICE candidates
The changes in `5c4c9d5` caused problems because we'd end up creating new identical pairs. For example, if an existing pair, `0-0`, exists as local: 192.168.1.33:5445, remote: 192.168.1.33:63461. T...
k0nserv created a branch on algesten/str0m

dedupe-remote-ice-candidates - A Sans I/O WebRTC implementation in Rust.

algesten created a comment on a pull request on algesten/str0m
Thanks!

View on GitHub

algesten pushed 1 commit to main algesten/str0m
  • Correctly handle per m-line Absolute Send Time 36ead9e

View on GitHub

algesten closed a pull request on algesten/str0m
Correctly handle per m-line Absolute Send Time
Related to #573 Remote peer might have [`Absolute Send Time`](http://www.webrtc.org/experiments/rtp-hdrext/abs-send-time) disabled for some m-line (like Firefiox doest not support abs-send-time ...
algesten created a comment on a pull request on algesten/str0m
Looks good! Just a lil lint problem.

View on GitHub

algesten created a review on a pull request on algesten/str0m

View on GitHub

alexlapa opened a draft pull request on algesten/str0m
Correctly handle per m-line Absolute Send Time
Related to #573 Remote peer might have [`Absolute Send Time`](http://www.webrtc.org/experiments/rtp-hdrext/abs-send-time) disabled for some m-line (like Firefiox doest not support abs-send-time ...
Load more