in logged in state city cannot be identified
Incident Report for Strawberry
Postmortem

https://strawberry-universe.slack.com/archives/C05PRP2FVPH/p1735288954724979

Copying from thread for documentation purpose

Josefine FlüggeJosefine Flügge  44 minutes ago
Not sure if other issues, but I reprouced the search errors locally and prod with contentful master, and located it to contentful requests for hotel content.This error:

[1] BadRequest: {
[1]   "status": 400,
[1]   "statusText": "Bad Request",
[1]   "message": "Response size too big. Maximum allowed response size: 7340032B.",

is due to that all entries of the hotel content type is asked for and the amount of data per request is too large so you have to divide the request into smaller batches

10:01

Idk if any web people from book are working but otherwise I can update that

Jesper ÅkerlindJesper Åkerlind  

42 minutes ago
All web people in book are off today unfortunately. If you're working and is able to help out that would be amazing @Josefine Flügge

Josefine FlüggeJosefine Flügge  41 minutes ago
Absolutely

Valeria Viana GusmaoValeria Viana Gusmao  36 minutes ago
@Josefine Flügge I'm around if you need me, I'll google around and see if there's anything we could do but reducing the nestedness, but other than that sounds like you're on it, right?

Josefine FlüggeJosefine Flügge  32 minutes ago
Thanks! Yeah,  what do you mean with reducing nestedness? The fix I thought of would be to reduce "limit", number of pages per request not levels included. I think this is the quickest fix right now that seems to do the trick but we should definitely check if we can improve the strategy overall with similar requests in the future (as it happened now "randomly") (edited) 

10:14

it's being deployed right now so let's check in some minutes

Valeria Viana GusmaoValeria Viana Gusmao  27 minutes ago
Yeah that’s even better @Josefine Flügge ! I meant to check if there are properties that we don’t need to fetch etc. Is the issue that we fetch all of the hotels at once?

Josefine FlüggeJosefine Flügge  23 minutes ago
Oh yeah now when I read it again yeah limiting the levels included would also decrease the size ofc

@valeriavgHmm my guess is that some data was added recently to one or more hotels in ctf so that the previous batch size (150 hotel entries per request) became too large

Posted Dec 27, 2024 - 10:45 CET

Resolved
when the user is in the logged in state the field to search for city was giving an error
Posted Dec 27, 2024 - 09:30 CET