archives/snowflake
2
0
Fork 0
mirror of https://gitlab.torproject.org/tpo/anti-censorship/pluggable-transports/snowflake.git synced 2025-10-13 20:11:19 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

webrtc.NewIceServer already does this split / validation

Browse source 
It also expects to be called with a non-empty string, which isn't
the case with this Split which always envokes the loop at least
once.  If -ice was omitted, an error was logged.
This commit is contained in:
Arlo Breault 2017-10-18 11:13:25 -04:00
parent 58556dc07b
commit bbb5a6aca2
1 changed files with 2 additions and 5 deletions
Download patch file Download diff file Expand all files Collapse all files

7
client/snowflake.go
View file

@ -145,11 +145,8 @@ func main() {
log.Println("\n\n\n --- Starting Snowflake Client ---") log.Println("\n\n\n --- Starting Snowflake Client ---")
var iceServers IceServerList var iceServers IceServerList
log.Println("IceServerList:") if len(strings.TrimSpace(*iceServersCommas)) > 0 {
for _, server := range strings.Split(*iceServersCommas, ",") { option := webrtc.OptionIceServer(*iceServersCommas)
// TODO: STUN / TURN url format validation?
log.Println(server)
option := webrtc.OptionIceServer(server)
iceServers = append(iceServers, option) iceServers = append(iceServers, option)
} }