WIP: Make dns, post_up and post_down optional

[nsg]

Proposed Changes

Like we discussed in #14. I have never worked with Home Assistant Addons before so I hope I got everything correct. My initial thought was to make dns, post_up and post_down optional via the schema, changing str to str? in the schema but that just used the default value from options instead. I changed approach to just allow an empty value instead.

DNS

Set the dns list to an empty list, dns: [], this will remove DNS = ... from wg0.conf

PostUP and PostDown

Set the post_up (or post_down) to an empty string, post_up: "", this will remove PostUp = ... from wg0.conf

Related Issues

• Need to create multiple peers in wireguard client under home assistant #14

---

What do you think? An alternative would to use str? and remove the default values for dns, post_up and post_down from options. That should not break existing installations, but will change the behavior a little. What do you prefer?

[bigmoby]

Hi @nsg is this PR in WIP or could I review it?

[nsg]

I did some more experimenting last evening and opened up a PR #16 to share. I also implemented an alternative to this PR with this commit nsg@414f2f9 ... this implements my above mentioned initial thought. I think that's closer to what a user expects, "remove a line of yaml = line removed from wg0.conf" ... this has of course the drawback that we can't use default values for these options, I prefer this solution.

What do you think?

---

Code samples

How to remove options in this PR:

interface:
  private_key: sEcrEtpRIvatEkey=
  address: 10.0.0.2/24
  dns: []
  post_up: ""
  post_down: ""

How to remove the same options in #16

interface:
  private_key: sEcrEtpRIvatEkey=
  address: 10.0.0.2/24

[bigmoby]

Hi @nsg thank you so much for your contribute 🙏
In these days I saw both PRs you left WIP and I get several hints from your code. I create the 0.1.9 version of this add-on, I hope you could enjoy it! 👍
Many thanks,
Bigmoby