Skip to content

Commit 0d2d183

Browse files
committed
resend_email_confirmation is allowed for all, don't leak data
1 parent 887404a commit 0d2d183

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

exordos_core/user_api/iam/api/controllers.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -288,7 +288,8 @@ def disable_otp(self, resource, password):
288288
def resend_email_confirmation(self, resource):
289289
app_endpoint = _get_app_endpoint(req=self._req)
290290
resource.resend_confirmation_event(app_endpoint=app_endpoint)
291-
return resource
291+
# Don't leak user data
292+
return None
292293

293294
@actions.post
294295
def force_confirm_email(self, resource):
@@ -297,7 +298,7 @@ def force_confirm_email(self, resource):
297298
raise iam_e.CanNotUpdateUser(uuid=resource.uuid, rule=rule)
298299

299300
resource.confirm_email()
300-
return resource
301+
return None
301302

302303
@actions.post
303304
def confirm_email(self, resource, code=None):

0 commit comments

Comments
 (0)