From 6ae7e49522e34b32fa317756b906a784317ac829 Mon Sep 17 00:00:00 2001 From: Andrew Morgan <1342360+anoadragon453@users.noreply.github.com> Date: Thu, 30 May 2019 12:30:43 +0100 Subject: [PATCH 1/5] Clarify what the client should receiving on email resending It was a little unclear what the client should do when asked to send a validation email and provides a `send_attempt` value that is not greater than previous attempts. As this is intended to be for when a client mistakenly sends the request twice, it makes logical sense that the implication was to simply resend a success value so the client doesn't error even when an email may have been sent on the first attempt. This behaviour was also mimicked in Synapse/Sydent. --- api/identity/definitions/request_email_validation.yaml | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/api/identity/definitions/request_email_validation.yaml b/api/identity/definitions/request_email_validation.yaml index 1194a18e..d6606f03 100644 --- a/api/identity/definitions/request_email_validation.yaml +++ b/api/identity/definitions/request_email_validation.yaml @@ -39,7 +39,8 @@ properties: avoid repeatedly sending the same email in the case of request retries between the POSTing user and the identity server. The client should increment this value if they desire a new - email (e.g. a reminder) to be sent. + email (e.g. a reminder) to be sent. If they do not, the server + should return a success but not resend the email. example: 1 next_link: type: string From c0c462999374a51b8430b88f7abe58a5a2b4386b Mon Sep 17 00:00:00 2001 From: Andrew Morgan <1342360+anoadragon453@users.noreply.github.com> Date: Thu, 30 May 2019 12:31:26 +0100 Subject: [PATCH 2/5] Slight word change --- api/identity/definitions/request_email_validation.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/api/identity/definitions/request_email_validation.yaml b/api/identity/definitions/request_email_validation.yaml index d6606f03..5d1aac8a 100644 --- a/api/identity/definitions/request_email_validation.yaml +++ b/api/identity/definitions/request_email_validation.yaml @@ -40,7 +40,7 @@ properties: retries between the POSTing user and the identity server. The client should increment this value if they desire a new email (e.g. a reminder) to be sent. If they do not, the server - should return a success but not resend the email. + should response with success but not resend the email. example: 1 next_link: type: string From dda8976d2544a22cb280fce243d23107e2bd4c88 Mon Sep 17 00:00:00 2001 From: Andrew Morgan Date: Thu, 30 May 2019 12:33:29 +0100 Subject: [PATCH 3/5] Add changelog --- changelogs/client_server/newsfragments/2057.clarification | 1 + 1 file changed, 1 insertion(+) create mode 100644 changelogs/client_server/newsfragments/2057.clarification diff --git a/changelogs/client_server/newsfragments/2057.clarification b/changelogs/client_server/newsfragments/2057.clarification new file mode 100644 index 00000000..de72c201 --- /dev/null +++ b/changelogs/client_server/newsfragments/2057.clarification @@ -0,0 +1 @@ +Clarify what the client should receive upon sending an identical email validation request multiple times. From 0b42a17352356eb879ee21f264dddfdcc78ef647 Mon Sep 17 00:00:00 2001 From: Andrew Morgan <1342360+anoadragon453@users.noreply.github.com> Date: Mon, 3 Jun 2019 09:48:38 +0100 Subject: [PATCH 4/5] Update api/identity/definitions/request_email_validation.yaml Co-Authored-By: Travis Ralston --- api/identity/definitions/request_email_validation.yaml | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/api/identity/definitions/request_email_validation.yaml b/api/identity/definitions/request_email_validation.yaml index 5d1aac8a..5f15bd41 100644 --- a/api/identity/definitions/request_email_validation.yaml +++ b/api/identity/definitions/request_email_validation.yaml @@ -40,7 +40,7 @@ properties: retries between the POSTing user and the identity server. The client should increment this value if they desire a new email (e.g. a reminder) to be sent. If they do not, the server - should response with success but not resend the email. + should respond with success but not resend the email. example: 1 next_link: type: string From 49b9bfc9f6ed5041807d8d893a29f9753381ad06 Mon Sep 17 00:00:00 2001 From: Andrew Morgan Date: Mon, 3 Jun 2019 09:48:53 +0100 Subject: [PATCH 5/5] Move changelog to the right place --- .../newsfragments/2057.clarification | 0 1 file changed, 0 insertions(+), 0 deletions(-) rename changelogs/{client_server => identity_service}/newsfragments/2057.clarification (100%) diff --git a/changelogs/client_server/newsfragments/2057.clarification b/changelogs/identity_service/newsfragments/2057.clarification similarity index 100% rename from changelogs/client_server/newsfragments/2057.clarification rename to changelogs/identity_service/newsfragments/2057.clarification