This change makes it easier for us to write tests with mock transports

Also removed some unnecessary code

Switch to containing all communication between the proxy and the broker
in the HTTP response body. This will make things easier if we ever use
something other than HTTP communicate between different actors in the
snowflake system.

Other changes to the protocol are as follows:
- requests are accompanied by a version number so the broker can be
backwards compatable if desired in the future
- all responses are 200 OK unless the request was badly formatted

And simplify EOF check.

Trac: 31028

The variable maxNumClients was unused, while connectionsPerClient was
used for spawning multiple proxyPairs. The former is a more appropriate
name for the multiplexing behaviour we use it for.

Multiplexing now just works thanks to implementing ticket #31310.

Increase readability of code a bit, the function descriptions were
automatically placed in the constructor when we moved from coffeescript.

Use their existence in the proxy pair list to indicate they are active.

We don't need it, and already have a function webrtcIsReady that tells
us what we need to know (whether a datachannel was opened before the
timeout period).

The only place it was used was in window.onpageunload, and we have a
better way of determining if the proxy is active there (through the ui).

I also removed that code from the webextension since the proxy won't
stop running unless you close the browser and after testing it looks
like that code doesn't notify the user anyway.

David Fifield authored 5 years ago and Cecylia Bocovich committed 5 years ago

https://bugs.torproject.org/32129

We haven't implemented SetDeadline for webRTCConn and the error messages
are misleading to proxy-go operators.

Use npm cldr package to get the language name that corresponds to the
country code for the language switcher

- cut down on size of bootstrap.css file
- remove unecessary styles
- fixed typo in javascript comment

Also modified the styling of the page to match the main tp.o page a bit
more

Right now we use the navigator language to determine localization and
replace the website contents with translated strings.

This will make it easier to have translated copies of the site in
subdirectories

Switched to using messages.json for translation strings for
snowflake@tp.o

Not closing peer connections was causing UDP sockets to remain open
indefinitely (as reported in ticket #31285).

A failure to set the git tag returns and undoes the changes done
previously

Added a new script to package the webextension. This will automatically
build and zip the source code and the webextension for upload. It take a
version as an argument and then checks the version in the manifest, and
locally commits a version bump.

The requirement to do so is obsolete and has already been removed from
other pluggable transports.

https://bugs.torproject.org/32046

With,
convert -background transparent toolbar-off.svg -define icon:auto-resize=32 toolbar-off.ico

.ico files were created with,
convert -density 256x256 -background transparent toolbar-on.svg -define icon:auto-resize -colors 256 toolbar-on.ico

Trac: 31537

Thanks cohosh for helping debug this. Uploaded files need correct group
ownership.

Shane Howearth authored 5 years ago and Cecylia Bocovich committed 5 years ago

last was initialised twice (creating a shadow), the second time inside
a case statement. The second initialisation is removed, keeping the use
of last aligned to the isame style as its use other parts of the case
statement.

Shane Howearth authored 5 years ago and Cecylia Bocovich committed 5 years ago

- Error strings are no longer capitalized nor end with punctuation
- Alias import
- Remove extraneous initilisation code (No need to provide zero value
	for variables, because the compiler does that anyway)