fix: add return_to parameter to API spec of createRecoveryLinkForIden…

…tity
ory · Jan 24, 2024 · 6bab5d5 · 6bab5d5
1 parent ce6b11a
commit 6bab5d5
Show file tree

Hide file tree

Showing 5 changed files with 33 additions and 1 deletion.
diff --git a/internal/client-go/api_identity.go b/internal/client-go/api_identity.go
diff --git a/internal/httpclient/api_identity.go b/internal/httpclient/api_identity.go
diff --git a/selfservice/strategy/link/strategy_recovery.go b/selfservice/strategy/link/strategy_recovery.go
@@ -69,6 +69,8 @@ func (s *Strategy) PopulateRecoveryMethod(r *http.Request, f *recovery.Flow) err
 type createRecoveryLinkForIdentity struct {
 	// in: body
 	Body createRecoveryLinkForIdentityBody
+	// in: query
+	ReturnTo string `json:"return_to"`
 }
 
 // Create Recovery Link for Identity Request Body
@@ -156,7 +158,7 @@ func (s *Strategy) createRecoveryLinkForIdentity(w http.ResponseWriter, r *http.
 		}
 	}
 
-	if time.Now().Add(expiresIn).Before(time.Now()) {
+	if expiresIn <= 0 {
 		s.d.Writer().WriteError(w, r, errors.WithStack(herodot.ErrBadRequest.WithReasonf(`Value from "expires_in" must be result to a future time: %s`, p.ExpiresIn)))
 		return
 	}

diff --git a/spec/api.json b/spec/api.json
@@ -4379,6 +4379,15 @@
       "post": {
         "description": "This endpoint creates a recovery link which should be given to the user in order for them to recover\n(or activate) their account.",
         "operationId": "createRecoveryLinkForIdentity",
+        "parameters": [
+          {
+            "in": "query",
+            "name": "return_to",
+            "schema": {
+              "type": "string"
+            }
+          }
+        ],
         "requestBody": {
           "content": {
             "application/json": {

diff --git a/spec/swagger.json b/spec/swagger.json
@@ -934,6 +934,11 @@
             "schema": {
               "$ref": "#/definitions/createRecoveryLinkForIdentityBody"
             }
+          },
+          {
+            "type": "string",
+            "name": "return_to",
+            "in": "query"
           }
         ],
         "responses": {