search

h44z / wg-portal

WireGuard Configuration Portal with LDAP connection
https://wgportal.org/
MIT License
958 stars 127 forks source link

v2 image testing results #206

Open pasztorl opened 10 months ago

pasztorl commented 10 months ago

HI,

I'm testing the v2 image.

There is one wg interface configured in server mode.

  1. I set up an ip cidr in Edit interface -> Interface -> "IP Addresses" it not configured to the wg interface.

  2. So I set up a post-up hook to run ip command to assign IP address to the wg interface, but it seems the command not runned when I click to save.

  3. Int the Interface status for wg interface not shows the interface's IP address

  4. When I set up a peer with allowedIPs option the site saves it, but not set it up to the wg interface.

In the log I haven't seen any errors. This should be works, but I miss something? How can I debug it?

pasztorl commented 10 months ago

Edit interface -> Peer defaults not saves the IP Networks, Allowed IP Addresses, DNS Server, DNS Search Domains fields

HPPinata commented 10 months ago

Are you hitting space after typing them in? If I remember correctly the entrys in those fields should be highlighted in blue when they actually apply. This only happens if you hit space bar after typing them in (even if it's just one entry). Otherwise they are still "saved" in the sense that you will see them after reloading and opening the page again but they won't apply.

pasztorl commented 10 months ago

ahh that was the problem, thanks! continuing the test..

HPPinata commented 10 months ago

Potential improvement: @h44z maybe there should be a help text or a warning when saving if you have filled in fields that will not apply? I also had this issue and it took me around half an hour until I realized the user defined entrys were not supposed to look different from the prefilled ones and I was simply doing something wrong (I actually experimented with different separations like comma, semicolon, etc. until I realized what the correct method was). The concept of turning the entrys into "tags" that can be deleted in one click instead of having to manually highlight and delete them is actually quite nice and now that I understand it I really like it. But it might need to be communicated more clearly to novice users, because (from my experience) it is not a common UX trope to have to "confirm" even your last entry in a space separated list with a trailing whitespace.

I'm not a UI/UX guy, but clearly also not the only one who took a while to get the way those entries work.

Sarevok933219 commented 9 months ago

HI,

I'm testing the v2 image.

There is one wg interface configured in server mode.

  1. I set up an ip cidr in Edit interface -> Interface -> "IP Addresses" it not configured to the wg interface.
  2. So I set up a post-up hook to run ip command to assign IP address to the wg interface, but it seems the command not runned when I click to save.
  3. Int the Interface status for wg interface not shows the interface's IP address
  4. When I set up a peer with allowedIPs option the site saves it, but not set it up to the wg interface.

In the log I haven't seen any errors. This should be works, but I miss something? How can I debug it? Add the inability to log in to the UI admin panel if Postgres is used as the backend

h44z commented 9 months ago

Potential improvement: @h44z maybe there should be a help text or a warning when saving if you have filled in fields that will not apply? I also had this issue and it took me around half an hour until I realized the user defined entrys were not supposed to look different from the prefilled ones and I was simply doing something wrong (I actually experimented with different separations like comma, semicolon, etc. until I realized what the correct method was). The concept of turning the entrys into "tags" that can be deleted in one click instead of having to manually highlight and delete them is actually quite nice and now that I understand it I really like it. But it might need to be communicated more clearly to novice users, because (from my experience) it is not a common UX trope to have to "confirm" even your last entry in a space separated list with a trailing whitespace.

I'm not a UI/UX guy, but clearly also not the only one who took a while to get the way those entries work.

Nice suggestion. I need to think about how to implement this - I am not a UI/UX guy either ;)