deblan-mirror/abraunegg-onedrive
deblan-mirror/abraunegg-onedrive
1
0
Fork 0
mirror of https://github.com/abraunegg/onedrive synced 2024-05-17 05:06:55 +02:00
Code Issues Projects Releases Wiki Activity

Potentially resolve #2699

Browse source 
* Update docker.md to clarify where to install Docker from
* Potentially resolve #2699 - needs testing by reporter
This commit is contained in:
abraunegg 2024-04-29 09:22:14 +10:00
parent 0f0bc88ed7
commit e1e35faec3
2 changed files with 20 additions and 5 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

5
docs/docker.md
View file

@ -38,10 +38,11 @@ Additionally there are specific version release tags for each release. Refer to
## Configuration Steps
### 1. Install 'docker' on your platform
Install 'docker' as per your distribution platform's instructions if not already installed as per the instructions on https://docs.docker.com/engine/install/
Install Docker for your system using the official instructions found at https://docs.docker.com/engine/install/.
> [!CAUTION]
> If you are using Ubuntu, do not install Docker from your distribution platform's repositories as these contain obsolete and outdated versions. You *must* install Docker from Docker provided packages.
> If you are using Ubuntu or any distribution based on Ubuntu, do not install Docker from your distribution's repositories, as they may contain obsolete versions. Instead, you must install Docker using the packages provided directly by Docker.
### 2. Configure 'docker' to allow non-privileged users to run Docker commands
Read https://docs.docker.com/engine/install/linux-postinstall/ to configure the 'docker' user group with your user account to allow your non 'root' user to run 'docker' commands.

20
src/sync.d
View file

@ -2153,6 +2153,8 @@ class SyncEngine {
string OneDriveFileXORHash;
string OneDriveFileSHA256Hash;
ulong jsonFileSize = 0;
Item databaseItem;
bool fileFoundInDB = false;
// Download item specifics
string downloadItemId = onedriveJSONItem["id"].str;
@ -2205,8 +2207,6 @@ class SyncEngine {
// Does the file already exist in the path locally?
if (exists(newItemPath)) {
// file exists locally already
Item databaseItem;
bool fileFoundInDB = false;
foreach (driveId; onlineDriveDetails.keys) {
if (itemDB.selectByPath(newItemPath, driveId, databaseItem)) {
fileFoundInDB = true;
@ -2348,6 +2348,7 @@ class SyncEngine {
} else {
// Downloaded file does not match size or hash .. which is it?
bool downloadValueMismatch = false;
// Size error?
if (downloadFileSize != jsonFileSize) {
// downloaded file size does not match
@ -2356,6 +2357,7 @@ class SyncEngine {
addLogEntry("OneDrive API reported size: " ~ to!string(jsonFileSize), ["debug"]);
addLogEntry("ERROR: File download size mis-match. Increase logging verbosity to determine why.");
}
// Hash Error
if (downloadedFileHash != onlineFileHash) {
// downloaded file hash does not match
@ -2364,6 +2366,7 @@ class SyncEngine {
addLogEntry("OneDrive API reported hash: " ~ onlineFileHash, ["debug"]);
addLogEntry("ERROR: File download hash mis-match. Increase logging verbosity to determine why.");
}
// .heic data loss check
// - https://github.com/abraunegg/onedrive/issues/2471
// - https://github.com/OneDrive/onedrive-api-docs/issues/1532
@ -2390,10 +2393,21 @@ class SyncEngine {
// If the computed hash does not equal provided online hash, consider this a failed download
if (downloadedFileHash != onlineFileHash) {
// We do not want this local file to remain on the local file system as it failed the integrity checks
addLogEntry("Removing file " ~ newItemPath ~ " due to failed integrity checks");
addLogEntry("Removing local file " ~ newItemPath ~ " due to failed integrity checks");
if (!dryRun) {
safeRemove(newItemPath);
}
// Was this item previously in-sync with the local system?
// We previously searched for the file in the DB, we need to use that record
if (fileFoundInDB) {
// Purge DB record so that the deleted local file does not cause an online delete
// In a --dry-run scenario, this is being done against a DB copy
addLogEntry("Removing DB record due to failed integrity checks");
itemDB.deleteById(databaseItem.driveId, databaseItem.id);
}
// Flag that the download failed
downloadFailed = true;
}
}