Fix recovery_target_action validation in restore command

- Only write recovery_target_action to PostgreSQL config when value is 'promote' or 'shutdown'
- Add warning log for unsupported target-action values (prevents invalid config like '!')
- Fixes documentation build failure where '!' was being written to recovery_target_action

Author: moizpgedge

src/command/restore/restore.c

@@ -1571,10 +1571,17 @@ restoreRecoveryOption(const unsigned int pgVersion)
         {
             const StringId targetAction = cfgOptionStrId(cfgOptTargetAction);
 
-            if (targetAction != CFGOPTVAL_TARGET_ACTION_PAUSE)
+            if (targetAction == CFGOPTVAL_TARGET_ACTION_PROMOTE || targetAction == CFGOPTVAL_TARGET_ACTION_SHUTDOWN)
             {
                 kvPut(result, VARSTRZ(RECOVERY_TARGET_ACTION), VARSTR(strIdToStr(targetAction)));
             }
+            else if (targetAction != CFGOPTVAL_TARGET_ACTION_PAUSE)
+            {
+                LOG_WARN_FMT(
+                    "option '" CFGOPT_TARGET_ACTION "' value '%s' ignored, defaulting to '"
+                    CFGOPTVAL_TARGET_ACTION_PAUSE_Z "'",
+                    strZ(strIdToStr(targetAction)));
+            }
         }
 
         // Write recovery_target_timeline if set