Refactor some lower level functions in tde_keyring

Previously write_key_provider_info() was a bit of a "do everything"
function that had very different behavior depending on what parameters
was passed to it. This commit reworks it to a "dumb" function that just
writes the data without asking questions and have the callers take
responsibility for data validity.

This is to make it easier to validate the data in different ways
depending on the caller's needs without further complicating
write_key_provider_info().

This commit also renames the KeyringProviderXLRecord to
KeyringProviderFileRecord as that structure is used for more things than
only WAL now.

Author: AndersAstrand

contrib/pg_tde/src/access/pg_tde_xlog.c

@@ -74,7 +74,7 @@ tdeheap_rmgr_redo(XLogReaderState *record)
 
 	else if (info == XLOG_TDE_ADD_KEY_PROVIDER_KEY)
 	{
-		KeyringProviderXLRecord *xlrec = (KeyringProviderXLRecord *) XLogRecGetData(record);
+		KeyringProviderFileRecord *xlrec = (KeyringProviderFileRecord *) XLogRecGetData(record);
 
 		redo_key_provider_info(xlrec);
 	}
@@ -129,7 +129,7 @@ tdeheap_rmgr_desc(StringInfo buf, XLogReaderState *record)
 	}
 	if (info == XLOG_TDE_ADD_KEY_PROVIDER_KEY)
 	{
-		KeyringProviderXLRecord *xlrec = (KeyringProviderXLRecord *) XLogRecGetData(record);
+		KeyringProviderFileRecord *xlrec = (KeyringProviderFileRecord *) XLogRecGetData(record);
 
 		appendStringInfo(buf, "add key provider %s for %u", xlrec->provider.provider_name, xlrec->database_id);
 	}