Update yggdrasil to v0.3.2 #71
Conversation
terraform/rtmp-server/bootstrap.sh
Outdated
| @@ -162,7 +168,7 @@ cd ~ | |||
| until [[ `ifconfig tun1 >/dev/null 2>&1; echo $?` -eq 0 ]]; do | |||
| sleep 1 | |||
| done | |||
| echo -n `ifconfig tun1 | grep 'inet6 fd' | awk '{print $2'}` > ~/client-keys/rtmp_yggdrasil | |||
| echo -n `ifconfig tun1 | grep -E 'inet6 2[0-9a-fA-F]{2}:' | awk '{print $2}'` > ~/client-keys/rtmp_yggdrasil | |||
There was a problem hiding this comment.
Although i have never run into this issue, There is an assumption we are making that Yggdrasil will actually be on Tun1.
I found you can define the interface name it will be using
sudo sed -i "s/IfName: auto/IfName: ygg0/" /etc/yggdrasil.conf
That way instead of tun1 Yggdrasil will be on ygg0
PS: Great regex, going to port it over to prototype 👍
There was a problem hiding this comment.
Also general comment:
should move to "$(xxx)" instead of the `xx`
SC2006: Use $(..) instead of legacy ...
There was a problem hiding this comment.
Thanks for the comment re: SC2006
Opened a new issue to address the more general problem of needing shellcheck.
|
Thanks for the review @darkdrgn2k I have changed iface name to |
|
I changed this to v0.3.2 because they exposed the APIs that I need, so I no longer have a dependency on the DEBUG APIs now (which itself has a silly bug in v0.3.0) so I am able to remove my hack too. If this looks okay please merge. @darkdrgn2k |
|
looks good.. |
Resolves #70
Also updates to latest v0.3.2 release of yggdrasil.