search

SharebookBR / sharebook-backend

Projeto backend de código livre para o app Sharebook.
117 stars 75 forks source link

Bug no endpoint de recuperar senha. #167

Closed raffacabofrio closed 5 years ago

raffacabofrio commented 5 years ago

ForgetEmail está recebendo o e-mail como parâmetro na query String... e se o e-mail tiver alguns caracteres diferentes (mas válidos, como + ), a api não funciona.

viniciustr commented 5 years ago

Após investigação da mensagem de erro, encontrei o seguinte link: https://stackoverflow.com/questions/7739233/double-escape-sequence-inside-a-url-the-request-filtering-module-is-configured

Algumas respostas sugerem uma modificação no Web.config para tirar a validação de caracteres reservados que o IIS está fazendo, mas acho melhor não fazer dessa maneira.

Como o endpoint da API já é um POST, recomendo movermos o endereço de email para o corpo da requisição. Assim, o problema é contornado de maneira melhor. Ficaria algo bem simples:

{
  "email": "fulano+teste@gmail.com"
}

O único porém é que o front precisará ser modificado também. No entanto, mudança é pequena. Vamos fazer assim?

wantero commented 5 years ago

Por mim fechou. A parte do front pode deixar que implemento assim que o Backend estiver em dev.

raffacabofrio commented 5 years ago

Top. Eu também concordo. Bora nesse caminho então.

viniciustr commented 5 years ago

@wantero como o backend já está pronto, podemos mandar bala no front!