Skip to content

[th/libnm-8021x-empty-strings]

Thomas Haller requested to merge th/libnm-8021x-empty-strings into main

we have many string properties.

In almost all cases, we should not allow "" (and only allow NULL). The reason is, that

  • what's the meaning anyway? Does it mean the same as NULL?
  • you couldn't reasonably enter "" in nmcli

also, ifcfg-rh writer often (wrongly) uses svSetValueStr() which coerces "" to NULL, thus loosing the value set.

Anyway. In case of these 802-1x properties, having them set to an empty word makes no sense, and in fact lets supplicant fail. As we cannot break previously created profiles, normalize them.

#973 (closed)

Merge request reports