患者作成時、存在しない電話番号などで、SMS送信に失敗した場合でも登録自体はできてしまう

[halsk]

患者作成時にSMS送信に失敗した場合、本来Patientは作成されないべきだが、実際には登録できてしまう。

remote-patient-monitoring-api/src/lambda/patient.ts

Lines 260 to 269 in 6ce0bdd

	if (res.status !== "100") {
	console.log("SMS Failed");
	return {
	statusCode: 400,
	body: JSON.stringify({
	errorCode: "RPM00104",
	errorMessage: "User was created but sending SMS failed",
	}),
	};
	}

でエラーコードを一応返している。（実際には #156 と相まってCORSエラーになるが）

SMSで送るログインURLを作るには一度レコードを作らないと行けないので上記のようになっているが、本来であれば、エラーになった時点で作成したレコードを削除すべきである。

[kenichisak]

SMS を送信しない場合は、不正な電話番号であっても 検知・削除できないので このタイミングで Patient のレコードは削除しなくてもいいのではと個人的に思いました。

患者を削除する機能もあったと思いますし、別の機能として、電話番号を変更したり SMS を再送する機能がもしあれば問題ない気がします。