v0.15.4: ARCH-1 step 4 — extract MountManager (final ARCH-1 step) #47
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,69 @@ | ||
| using System; | ||
| using System.Diagnostics; | ||
| using System.IO; | ||
|
|
||
| namespace Pixelpipe | ||
| { | ||
| // ARCH-1 step 4 (v0.15.4, audit): fourth and final collaborator | ||
| // extracted from the TrayContext partial. Owns just the rclone mount | ||
| // process-lifecycle bits: argument assembly + Process.Start + Job Object | ||
| // binding. The dialog flow (rclone-missing, WinFsp-missing, remote-not- | ||
| // configured, drive-in-use, post-launch result) stays in TrayContext | ||
| // because it's heavily UI-coupled. | ||
| // | ||
| // This is intentionally a thin extraction. The audit calls ARCH-1 | ||
| // incremental; later releases can pull more orchestration in once the | ||
| // shape of "what doesn't need a TrayContext" stabilises. | ||
| internal sealed class MountManager | ||
| { | ||
| // Pure helper, tests cover it. Builds the rclone mount argv that | ||
| // Pixelpipe has used since v0.5.x with all the cache-mode, VFS, | ||
| // network-mode, RC, and bandwidth flags in one place. Caller passes | ||
| // RcCommonFlags + effective bandwidth so this class doesn't need to | ||
| // know about the auth token / per-profile bandwidth resolution. | ||
| internal static string BuildMountArgs(RemoteProfile p, bool fullCache, string rcCommonFlags, string effectiveBandwidth) | ||
| { | ||
| if (p == null) return ""; | ||
| string cacheMode = fullCache ? "full" : "writes"; | ||
| string args = "mount " + TrayContext.QuoteArg(TrayContext.NormalizeRemoteName(p.Remote)) | ||
| + " " + TrayContext.QuoteArg(TrayContext.NormalizeDriveLetter(p.DriveLetter)) | ||
| + " --links" | ||
| + (String.Equals(p.MountMode, "network", StringComparison.OrdinalIgnoreCase) ? " --network-mode" : "") | ||
| + " --vfs-cache-mode " + cacheMode | ||
| + " --dir-cache-time 10m" | ||
| + " --poll-interval 1m" | ||
| + " --vfs-write-back 10s" | ||
| + " --vfs-cache-max-age 6h" | ||
| + " --vfs-cache-max-size 5G" | ||
| + " --volname " + TrayContext.QuoteArg(p.Label) | ||
| + " --rc " + (rcCommonFlags ?? "") | ||
| + " --log-level INFO" | ||
| + " --log-file " + TrayContext.QuoteArg(p.LogFile); | ||
| if (!String.IsNullOrEmpty(effectiveBandwidth) | ||
| && !String.Equals(effectiveBandwidth, "off", StringComparison.OrdinalIgnoreCase)) | ||
| { | ||
| args += " --bwlimit " + effectiveBandwidth; | ||
| } | ||
| return args; | ||
| } | ||
|
|
||
| // Spawns rclone mount and binds the child to the kill-on-job-close | ||
| // Job Object so it dies with Pixelpipe even on Task Manager kill / | ||
| // crash / sign-out. Returns the started Process; caller is | ||
| // responsible for the post-launch monitor (1900 ms wait + exit | ||
| // check + UI feedback). | ||
| internal static Process StartMountProcess(string rclonePath, string args, Action<string> logJobWarn) | ||
| { | ||
| ProcessStartInfo psi = new ProcessStartInfo(); | ||
| psi.FileName = rclonePath; | ||
| psi.Arguments = args; | ||
| psi.UseShellExecute = false; | ||
| psi.CreateNoWindow = true; | ||
| psi.WindowStyle = ProcessWindowStyle.Hidden; | ||
| psi.WorkingDirectory = Environment.GetFolderPath(Environment.SpecialFolder.UserProfile); | ||
| Process child = Process.Start(psi); | ||
| RcloneJob.TryAssign(child, logJobWarn); | ||
|
Comment on lines
+64
to
+65
There was a problem hiding this comment. Defensive programming: Process child = Process.Start(psi);
if (child != null)
{
RcloneJob.TryAssign(child, logJobWarn);
} |
||
| return child; | ||
| } | ||
| } | ||
| } | ||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If
p.LogFileis null or empty, passing--log-file ""to rclone can cause it to fail to start on Windows due to an invalid path. It is safer to conditionally append the--log-fileargument only when a valid log file path is provided.