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

Android 14 tab jopplin app not sync via dropbox #10681

Closed
parduspanteri opened this issue Jul 4, 2024 · 4 comments · Fixed by #10759
Closed

Android 14 tab jopplin app not sync via dropbox #10681

parduspanteri opened this issue Jul 4, 2024 · 4 comments · Fixed by #10759
Assignees
Labels
bug It's a bug high High priority issues sync sync related issue upstream There's a problem with upstream code.

Comments

@parduspanteri
Copy link

Operating system

Android

Joplin version

3.0.7

Desktop version info

No response

Current behaviour

photo_2024-07-03_17-49-32

Expected behaviour

I expected Joplin to synchronize notes via Dropbox.

Logs

syncReport-1720077376871.txt

@parduspanteri parduspanteri added the bug It's a bug label Jul 4, 2024
@personalizedrefrigerator personalizedrefrigerator added sync sync related issue high High priority issues labels Jul 5, 2024
@personalizedrefrigerator
Copy link
Collaborator

personalizedrefrigerator commented Jul 8, 2024

I haven't been able to reproduce this. However, I've only tested with the English (UK) and Español (España) locales selected (in Joplin).

Here's what I've tried:

  • Set up Dropbox sync with encryption disabled on Android 13.
    • Joplin 3.0.8, built from the dev branch (development mode).
  • Created a small number of notes and verified that these notes can be successfully synced with Dropbox.
  • Connected and synced a desktop client.
  • Attached a large file from the Android device (174 MiB) and verified that it syncs successfully.

Things that could still be tested:

  • Sync with a larger note collection.
  • Sync with encryption enabled.
  • Test on an Android 14 emulator.
  • Test with a release version of Joplin.

The attached log contains Error: RNFetchBlob request error: Unexpected char 0x131 at 10 in header name: dropbox-apı-argnull, where the ı in dropbox-apı-argnull is a dotless i character. I suspect that this should be an ASCII i. As such, I think that this issue is related to localization-related logic.

A GitHub code search for /dropbox-apı/ in Joplin's code, however, returns zero results.

Follow-up questions:

  • What display language is the Android device set to globally?
  • What display language is Joplin set to (if different)?

@personalizedrefrigerator personalizedrefrigerator added the translation i18n and l10n label Jul 8, 2024
@parduspanteri
Copy link
Author

Tablet and joplin app in turkish language.

@personalizedrefrigerator
Copy link
Collaborator

personalizedrefrigerator commented Jul 8, 2024

Upstream issue: joltup/rn-fetch-blob#573.

We likely need to either switch to a fork of rn-fetch-blob or patch rn-fetch-blob to work around the issue. The upstream issue recommends switching to react-native-blob-util.

Joplin previously migrated to react-native-blob-util in this commit, but was migrated back to rn-fetch-blob in this commit to fix the sync issues reported in this forum post.

Edit 1: Reworded for clarity.
Edit 2: The upstream commit in the react-native-blob-util fork that fixes the issue seems to be RonRadtke/react-native-blob-util@46876cb.

@personalizedrefrigerator personalizedrefrigerator added upstream There's a problem with upstream code. and removed translation i18n and l10n labels Jul 8, 2024
@parduspanteri
Copy link
Author

Upstream issue: joltup/rn-fetch-blob#573.

We likely need to either switch to a fork of rn-fetch-blob or patch rn-fetch-blob to work around the issue. The upstream issue recommends switching to react-native-blob-util.

Joplin previously migrated to react-native-blob-util in this commit, but was migrated back to rn-fetch-blob in this commit to fix the sync issues reported in this forum post.

Edit 1: Reworded for clarity. Edit 2: The upstream commit in the react-native-blob-util fork that fixes the issue seems to be RonRadtke/react-native-blob-util@46876cb.

Based on what is described here, when I changed the language of the tablet to English and entered the joplin app, the synchronization worked. Then when I switched back to Turkish, the synchronization continued to work. The problem was temporarily solved. But this bug from 2020 still seems to continuing.Thank you for your help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug It's a bug high High priority issues sync sync related issue upstream There's a problem with upstream code.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants