search

cofacts / rumors-api

GraphQL API server for clients like rumors-site and rumors-line-bot
https://api.cofacts.tw
MIT License
111 stars 26 forks source link

Internal server error when logging in in search result #37

Open MrOrz opened 7 years ago

MrOrz commented 7 years ago

Steps to reproduce:

  1. Go to article list page. If already logged in, logout first.
  2. In search box type in Chinese characters, perform search
  3. Login using any method
  4. sees internal server error.

Root cause: URLs are not encoded, but redirect location requires so.

MrOrz commented 4 years ago

Should be solved since URL is now has MAX_KEYWORD_LENGTH https://github.com/cofacts/rumors-site/commit/a938503002fdf8474e15d479bf49103e86840744

MrOrz commented 4 years ago

Oops, it's irrelevant of length. reopen.