Skip to content

Commit e8007dc

Browse files
committed
Update exception message in GetVideosFromModuleAsync for clarity on usage
1 parent faf1149 commit e8007dc

File tree

2 files changed

+6
-1
lines changed

2 files changed

+6
-1
lines changed

README.md

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -224,6 +224,11 @@ dotnet run
224224
- Client verification (PIN via SMS) is often enabled. This is normal behavior.
225225
- The Blink API can be unstable without pauses between requests — use `GeneralSleepTime`.
226226

227+
Note on clip IDs (observed behavior):
228+
229+
- The IDs returned in the video list appear to be short‑lived session identifiers. In practice, fetching the binary clip by a previously listed ID may start failing with “file not found” after a short time window (approx. 10–20 minutes, based on observations; not officially documented).
230+
- Recommendation: process videos in small batches. Fetch a list, immediately download a subset, then re‑list before continuing to avoid stale IDs.
231+
227232
## Security
228233

229234
- Do not store login/password in the repository. Use user secrets, environment variables, or encrypted stores.

Sources/Blink/BlinkClient.VideoMethods.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ public async Task<IEnumerable<BlinkVideoInfo>> GetVideosFromSingleModuleAsync()
6060
var dashboard = await GetDashboardAsync();
6161
if (dashboard.SyncModules.Length > 1)
6262
{
63-
throw new BlinkClientException("More than one sync module found. Use GetVideosAsync() instead.");
63+
throw new BlinkClientException("More than one sync module found. Use GetVideosFromModuleAsync(SyncModule module) instead.");
6464
}
6565
var module = dashboard.SyncModules.SingleOrDefault()
6666
?? throw new BlinkClientException("No sync modules found");

0 commit comments

Comments
 (0)