-
Notifications
You must be signed in to change notification settings - Fork 10
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
No sensor data is being collected when using a self-hosted server #31
Comments
Adithya, I hope Collaborator is not using the play-store - beta build. Now beta version is 1.1. |
Does the beta fix these issues? It works on my device but the sensor collection service notification disappears after some time. Does that mean there is no data being collected? |
Aditya, Can you share the device information. We are not facing such issues, if possible I will share a build will log file so we can track it. |
The collaborator is running his own separate infrastructure so I do not have access to that info on his end. However, I am testing on both a Pixel 2 XL (Android 10) and a Pixel 4 XL (Android 11). |
Ok Adithya, we haven't tested in android 11. I can share a test build with log file enabled is that okay. |
Yes, that's fine. A few questions, though:
|
1 - Goto internal-storage | Android | data | digital.lamp.mindlamp | files | LampLog.txt |
@amaljofyzco If you enable it, though, it should not impact the performance of the app or sensor collection. We should find a way to always save logs with confidential information stripped out and remotely request them for diagnostic purposes. Do you have any ideas on an implementation for this? |
@avaidyam Can we update beta to production and test that and see if we get these issues ? We don't want to spend time on it if its a build issue. In order to avoid confusion it would be better if we can just update the production with the current beta and test it again. As beta has fixes data collection which is not there in production. |
Hi @amaljofyzco can you confirm the 1.1 is up to date with what is updated to beta ? |
@amaljofyzco there is a way to send log files to developer account from a separate thread in Android. We can implement that and update the build asap. I have shared the info with you in chat. |
@amaljofyzco suggested we can integrate firebase analytics to collect the logs from the app. We will update the build with that integrated asap to figure out what's going on with Android 11 |
why was it closed :) |
Let's keep this issue open until that integration is complete and the collaborator reports to us that the issue is fixed on their end. |
Aditya, could you add me as a member in your firebase account (lampv2test@gmail.com). So that I can access it for setting up the analytics. |
sorry by mistake |
@amaljofyzco I've added you to the Firebase account temporarily - let me know what changes you've made when you're done so we can document them too! |
@amaljofyzco Can you remind me why we are enabling analytics (this requires enabling Google Analytics) for the app? I thought Crashalytics was what we wanted? |
Adithya, I also added the event-logs in the app. Like when we are sending the sensor-data we will post an event. Crashlytics also implemented, I will be updating the build after testing. Thanks |
Can you clarify exactly which new event logs you have added? |
Adithya, The event - logs added are follows.
Crashlytics also enabled. |
By event logs are you suggesting these are |
No SensorEvent object is passing to logs. Only the size of sensor object list is passing as a string. |
I have updated my pixel 3xl to Android 11, a small Fix done inside our framework code, which doesn't support in 11OS. |
@amaljofy So the items you mentioned are being logged, not uploaded to the LAMP API? If so, that's helpful - thanks! |
Correct, I will be pushing to beta shortly. |
That's fantastic - thanks @amaljofy! I'll have my team also use the beta and investigate issues on our end, and if we find anything we'll respond here. |
Code check-in done. App updated to playstore-beta, now its Being reviewed. |
@amaljofy @michaelmenon Another recent report:
I think this is turning into a very urgent issue that we need to resolve quickly. |
@jijopulikkottil Let me invite the collaborator to respond here! //cc @nerfanian @zone86 |
@jijopulikkottil @michaelmenon I’ve just shared a collaborator’s iOS device logs privately with you on Basecamp. Please take a look and let me know if that issue has been fixed. |
We are doing some optimisations on our code.. its still testing mode. Once we pushed the code, hope we can provide them the testflight beta build to test sensor data syncing issue. |
@kkadapa Were you noticing this issue on iOS or Android or both? |
Hi @avaidyam I see this issue currently only for iOS as mentioned here https://mindlamp.discourse.group/t/question-regarding-the-couchdb-step-count-bug/262 |
Hi @avaidyam I installed TestFlight version 2021.1.21(237) and allowed to sync my Steps but I still don't see Steps data in Prevent tab as you can see below, please let me know if you need any additional details |
@kkadapa You will not be able to see raw step count data in the dashboard; you have to use the API to access it for now. |
@kkadapa I've investigated this issue further to ensure we're able to solve the problem. After logging in to the latest production release of the mindLAMP app on an iPhone 6s using the credentials you specified in our email conversation, I performed a man-in-the-middle attack to intercept and inspect the incoming and outgoing data between the app and your self-hosted server. This issue does not seem to be with the app or the server code itself. You/your organization will need to reconfigure the load balancer currently serving the API Server as it currently has a maximum payload size limit that is far too low (< 1 MB). All requests from the app are being returned with a Below is the command I executed to confirm this issue (replace
Below is the fake data I generated and used: The response from your load balancer was: <html>
<head><title>413 Request Entity Too Large</title></head>
<body bgcolor="white">
<center><h1>413 Request Entity Too Large</h1></center>
<hr><center>nginx/1.14.0 (Ubuntu)</center>
</body>
</html> @jijopulikkottil @amaljofy @michaelmenon |
@amaljofyzco Do you know if this/related sensor issues are fixed in v1.1 since you pushed that last week?
This issue has been reported multiple times across both iOS and Android devices.
The text was updated successfully, but these errors were encountered: