[openstreetmap/openstreetmap-website] Use where instead of find to prevent 404 (b9c85c2)
Kamil Monicz
notifications at github.com
Sat Aug 26 15:16:53 UTC 2023
> Did you expect 404 when one user was missing or when all of them were?
Previous logic worked this way:
* Return 404 if at least one user is missing
While a very weird behavior indeed, I wrote my code around this issue. I have seen worse things in my life.
The current logic works this way:
* Always return 200 and skip not found users
My previously coded applications are not ready for such change.
The green lines are additions I had to do today when I noticed my app stopped working:

--
Reply to this email directly or view it on GitHub:
https://github.com/openstreetmap/openstreetmap-website/commit/b9c85c269726faad3c2613b37f01683c8b59f5c6#commitcomment-125594991
You are receiving this because you are subscribed to this thread.
Message ID: <openstreetmap/openstreetmap-website/commit/b9c85c269726faad3c2613b37f01683c8b59f5c6/125594991 at github.com>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.openstreetmap.org/pipermail/rails-dev/attachments/20230826/3c55c90b/attachment.htm>
More information about the rails-dev
mailing list