- Jan 30, 2020
-
-
David Fifield authored
This makes the return type satisfy the io.ReadWriteCloser interface directly.
-
David Fifield authored
Rename websocketconn.WebSocketConn to websocketconn.Conn, and websocketconn.NewWebSocketConn to websocketconn.New Following the guidelines at https://blog.golang.org/package-names#TOC_3%2e
-
David Fifield authored
Only once, not again on every call to initServer.
-
David Fifield authored
This was the only case out of the three not to show it.
-
- Jan 29, 2020
-
-
Cecylia Bocovich authored
We are no longer checking for nil BrokerChannels in Catch because this case is caught from the return values of NewBrokerChannel. This change caused a no longer necessary unit test to hang.
-
Cecylia Bocovich authored
We changed NewBrokerChannel to return an error value on failure. This updates the tests to check that value.
-
- Jan 28, 2020
-
-
David Fifield authored
-
David Fifield authored
The call was copyLoop(socks, snowflake) but the function signature was func copyLoop(WebRTC, SOCKS io.ReadWriter) { The mistake was mostly harmless, because both arguments were treated the same, except that error logs would have reported the wrong direction.
-
David Fifield authored
-
David Fifield authored
Instead of returning nil from NewBrokerChannel and having WebRTCDialer.Catch check for nil, let NewBrokerChannel return an error and bail out before calling WebRTCDialer.Catch. Suggested by cohosh. https://bugs.torproject.org/33040#comment:3
-
- Jan 23, 2020
-
-
David Fifield authored
Closing the channel makes it always immediately selectable.
-
David Fifield authored
-
David Fifield authored
-
David Fifield authored
-
David Fifield authored
-
David Fifield authored
-
- Jan 21, 2020
-
-
David Fifield authored
These functions are called for their side effect of sending a PT error message on stdout; they also return a representation of the error message as an error object for the caller to use if it wishes. These functions *always* return a non-nil error object; it is not something to be logged, any more than the return value of errors.New is. The mistaken logging was added in https://bugs.torproject.org/31794 b26c7a7a 3ec9dd19 ed3d42e1
-
- Dec 13, 2019
-
-
Jascha authored
-
- Dec 06, 2019
-
-
Arlo Breault authored
Trac: 32499
-
Arlo Breault authored
Trac: 32499
-
Cecylia Bocovich authored
-
Cecylia Bocovich authored
There were a few tests that needed refreshing since the introduction of the pion library. Also added a few tests for the ICE server parsing function in the client.
-
Cecylia Bocovich authored
Now tests the proxy and initServer functionalities. The tests use the same websocket library as the server and proxy-go implementations.
-
- Dec 05, 2019
-
-
Cecylia Bocovich authored
Added another lock to the metrics struct to synchronize accesses to the broker stats. There's a possible race condition if stats are updated at the same time they are being logged.
-
Cecylia Bocovich authored
We had some data races in the broker that occur when proxies and clients modify the heap/snowflake map at the same time. This test has a client and proxy access the broker simultaneously to check for data races.
-
Cecylia Bocovich authored
There's a race condition in the broker where both the proxy and the client processes try to pop/remove the same snowflake from the heap. This patch adds synchronization to prevent simultaneous accesses to snowflakes.
-
- Dec 03, 2019
-
- Nov 28, 2019
-
-
Cecylia Bocovich authored
-
Cecylia Bocovich authored
-
Cecylia Bocovich authored
-
Cecylia Bocovich authored
This allows badge and standalone proxies to tell the broker what proxy type they are.
-
Cecylia Bocovich authored
Proxies now include information about what type they are when they poll for client offers. The broker saves this information along with snowflake ids and outputs it on the /debug page.
-
- Nov 22, 2019
-
-
Arlo Breault authored
-
- Nov 21, 2019
-
-
Arlo Breault authored
Trac: 32465
-
- Nov 13, 2019
-
-
Cecylia Bocovich authored
-
Cecylia Bocovich authored
-
Cecylia Bocovich authored
-
Cecylia Bocovich authored
This change makes it easier for us to write tests with mock transports
-
Cecylia Bocovich authored
-
Cecylia Bocovich authored
Also removed some unnecessary code
-