-
-
Notifications
You must be signed in to change notification settings - Fork 4.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Server error: Parameters passed into interpolateUrl must be strings #6158
Comments
Hello @nithusha21 @DubeySandeep, I was confusing that did function call after every 5 minutes (when user login) or call after every 5 minutes (when the user starts an exploration ) ? And if it calls after every 5 minutes when user login then this will always occur because this function calls |
@import-keshav, Thanks for looking into it,
StatsReportingService gets initialized only when users start playing exploration (over here). Once a user leaves an exploration the tab gets refreshed and the $interval function gets cleared. Also, this $ interval doesn't get called in the I would suggest you to try reproducing this locally (If you think this issue caused by whatever you have said above) and add steps to reproduce, and after that, you can go further ahead and look for the fix. |
Hi @import-keshav, Any updates on this issue ? We need to fix these user-facing bugs soon! Let us know if you are stuck somewhere, we'll be happy to help you out. Thanks! |
Sorii @DubeySandeep @bansalnitish, for not giving any updates.
then the assumption I was assuming above was incorrect. So currently i was investigating it and give updates ASAP thanks! |
Hello @bansalnitish @DubeySandeep, I investigate further for the issue and I stuck somewhere so please help me.
But since the 1st possibility is not there because if the And the reason for the second possibility is maybe some other services were using this service( |
@import-keshav, Thanks for looking into it.
Can you please look for the place where we use to generate exp_id? I don't think this will be possible cause.
I'm not sure about how you can reproduce this issue, but I agree that it can happen when we will be running |
Thanks @DubeySandeep for the response.
yes, you are right, that's can't possible that's why I also mentioned that this is not a reason for this. |
And one more thing to ask, as @DubeySandeep told
so instead of checking it, did I rearrange scripts import? It would not create any code break (I think) |
Hi @import-keshav, Thanks for looking into the issue. For your query above:
I would suggest to check what @DubeySandeep has suggested. Also, try to draw a sequence diagram of what is actually happening to reproduce the issue (You can refer this comment for making a sequence diagram) . We need to have a clear idea of why this issue happens instead of just adding a fix without finding the actual cause. |
Thanks @bansalnitish for the response. |
Here is my analysis for this server error: https://docs.google.com/document/d/1IZ4lN7EjatnbwL6qWtOdY0V1Sp2ZLV9N02FuhzhVD6Q/edit?usp=sharing |
This error occured recently, here is the latest stack trace:
|
I was able to reproduce this locally. This would happen on a slow connection where the $interval sets off before the initSession function is called. I was able to reproduce this by tweaking the $interval duration to a shorter duration and throttling my connection speed. I propose that we extract the $interval into it's own function and call it inside initSession. Since, it does not make sense to start a 5 minute timer before the page has completely loaded and exploration initialized. @DubeySandeep, @nithusha21 what are your thoughts? |
This error occurred with a different stacktrace:
Reopening. |
This error was seen on the production server. It was first reported recently, and might be due to some code change to these files that was done recently.
The text was updated successfully, but these errors were encountered: