Tunnel Auth Verifiers config adds extra default config entry after restart

• Type: Regression Bug
• Status: Closed
• Resolution: Fixed
• Affects Version/s: Master
• Fix Version/s: 7.0.0 DXP FP58, 7.0.0 DXP SP9, 7.0.X, 7.1.0 M2, 7.1.X, Master
• Component/s: Application Security > Auth Verifiers, Portal Services > Configuration Management
• Labels:
  • liferay-dxp-70-sp9
  • liferay-fixpack-de-58-7010
  • profiq
• Branch Version/s:
  7.0.x
• Backported to Branch:
  Committed
• Fix Priority: 4
• Git Pull Request: https://github.com/brianchandotcom/liferay-portal/pull/56289

In case user wants to setup verification of Tunnel Authentication he needs to modify Tunnel Auth Verifier config. To make setup permanent (or to apply setup) he needs to restart web container. After restart, however, an extra default entry is added to the list of Tunnel Authentication Verifiers configurations resulting in two entries where there should only be the one that contains the modified values.

Steps to reproduce:

1. Navigate to Control Panel > System settings > Foundation > Tunnel Auth Verifiers
2. Add remote ip (eg. 192.168.11.7) to the Hosts allowed
3. Click update button
4. Refresh page to make sure that changes were saved
5. Reboot liferay
6. Navigate to Control Panel > System settings > Foundation > Tunnel Auth Verifiers

Expected result: IP added in step 2 is shown and there is no extra default configuration entry.
Actual result: The changes to the setting entry are saved (adding an IP in step 2). However, a separate entry shows up with all of the default settings. This results in having two entries, one with the default settings and one with the changes the user made.

Reproduced on:
Windows 10 + Tomcat 8.0.32 + MySQL 5.6 + master
7e514af3c1eabf6bf018799b538b2aba4cdbd983

Fix priority: S3+L4 = 4