Skip to content
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

Sync error on remote site - 'EOF while parsing' #3813

Closed
regotaina opened this issue May 7, 2024 · 12 comments · Fixed by #5998
Closed

Sync error on remote site - 'EOF while parsing' #3813

regotaina opened this issue May 7, 2024 · 12 comments · Fixed by #5998
Assignees
Labels
bug Something is borken Needs Investigation Not sure what's the cause of issue Severity: High Bugs breaking core functionality or with no/unacceptable workaround. High impact. Next patch release
Milestone

Comments

@regotaina
Copy link
Contributor

regotaina commented May 7, 2024

What went wrong? 😲

After doing any transaction on a remote site(android), I press 'sync now' and get the following error message:

Screenshot 2024-05-07 at 3 35 24 PM

the problem is resolved if I press the sync button for the 2nd time.

Expected behaviour 🤔

No need to press sync twice to get it to work.

How to Reproduce 🔨

Steps to reproduce the behaviour:

  1. Go to 'remote site (android)
  2. Do any transactions (internal order, outbound shipment, etc..)
  3. Go to sync and press the button 'sync now'
  4. See error

Your environment 🌱

  • Open mSupply Version: V2.0 RC3
  • Legacy mSupply Central Server Version: V7.14.0
  • Platform: android
@regotaina regotaina added bug Something is borken needs triage labels May 7, 2024
@DhanyaHerath DhanyaHerath added Severity: Normal Bugs which have an acceptable workaround. Moderate/tolerable user impact. Next minor release. and removed needs triage labels May 8, 2024
@DhanyaHerath
Copy link
Contributor

Triage: needs checking to reproduce

@roxy-dao
Copy link
Contributor

Update: Doesn't seem to be happening anymore and could have been something to do with set up. Have tested with problematic data and seems to be working fine in development

@roxy-dao roxy-dao closed this as not planned Won't fix, can't repro, duplicate, stale May 14, 2024
@nishaDangol-Sussol nishaDangol-Sussol added the Closed: Invalid Issue was closed because it is invalid, i.e. cannot replicate, not a bug, etc. label Jul 24, 2024
@jmbrunskill
Copy link
Contributor

Seems to happen more often straight after login, then gets better after 10min?

@marthakeezy
Copy link

marthakeezy commented Oct 14, 2024

Reopening since the issue is happening again for me and @regotaina (going to look at this with @kahn-msupply ). The problem is no longer resolved when you press Sync twice, I have to wait for about 10-15 minutes befor sync eventually works. Right not I've been waiting for 26 minutes. Tainá and I are on different datafiles too

I am on V2.3.0-RC8

Also changing severity to High since it's a major testing blocker, I cannot sync anything between my remote site and Central Server

@marthakeezy marthakeezy reopened this Oct 14, 2024
@marthakeezy marthakeezy added Severity: High Bugs breaking core functionality or with no/unacceptable workaround. High impact. Next patch release and removed Closed: Invalid Issue was closed because it is invalid, i.e. cannot replicate, not a bug, etc. Severity: Normal Bugs which have an acceptable workaround. Moderate/tolerable user impact. Next minor release. labels Oct 14, 2024
@regotaina
Copy link
Contributor Author

just clarifying, sync still works here - the error occurs sporadically, not following a pattern. It's not blocking
Like James mentioned, sync issues might be or not connected to this one.

@kahn-msupply
Copy link

I haven't been able to pin-point this issue. I will be doing some testing to rule out data file issues. I'll make sure to update here with my findings.

@jmbrunskill
Copy link
Contributor

@kahn-msupply have you confirmed it does or doesn't still happen with Caddy?

@kahn-msupply
Copy link

@kahn-msupply have you confirmed it does or doesn't still happen with Caddy?

Tests show this issue is occurring only with Caddy - I'll be doing some more testing to see if I can replicate this on another test server, and if there is an effect on remote sites.

@jmbrunskill
Copy link
Contributor

@triage team, adding this for triage again.

Reasoning is...

  1. We're rolling out Caddy to clients, which already have challenging internet connections. If this introduces additional issues with sync that won't help matters!
  2. This is causing significant interruption to QA testing as tests have to try to sync multiple times, I've seen it require 4-5 sync attempts to work.

Maybe needs some more refinement and time boxing but I think probably worth spending some time to investigate further.

Obviously one option is to just remove Caddy from the workflow?
#5080 might help with QA issues

@kahn-msupply
Copy link

kahn-msupply commented Oct 16, 2024

One interesting thing I have noticed (from Desktop application, installed next to Oepn mSupply Central Server)... I have 3 options available. Server is running on Port 443. AppData was cleared and options persist. Netstat -ano shows no PID attached to ports 8000 (SSL Certs / Manual) and 7000 (Caddy Reverse Proxy). I'll keep providing updates.

Screenshot 2024-10-15 at 5 16 45 PM

@marthakeezy
Copy link

I always have 2 options shown - 7000 and 8000. Only one of them works though

@andreievg andreievg added Needs Investigation Not sure what's the cause of issue and removed needs triage labels Oct 16, 2024
@andreievg andreievg added this to the V2.4.0 milestone Oct 16, 2024
@lache-melvin lache-melvin added the Team Piwakawaka James, Carl, John, Zachariah label Oct 30, 2024
@roxy-dao roxy-dao modified the milestones: V2.4.0, v2.4.0-RC2, V2.4.0-RC3 Nov 13, 2024
@marthakeezy
Copy link

Happening again in V2.5.0-reports2

@andreievg andreievg removed the Team Piwakawaka James, Carl, John, Zachariah label Jan 6, 2025
@andreievg andreievg self-assigned this Jan 6, 2025
@andreievg andreievg modified the milestones: Refactor fortnight, v2.6.0 Jan 6, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something is borken Needs Investigation Not sure what's the cause of issue Severity: High Bugs breaking core functionality or with no/unacceptable workaround. High impact. Next patch release
Projects
None yet
Development

Successfully merging a pull request may close this issue.

9 participants