[Bug]: Sync recording lineage lacks limit clamping and origin ID validation
Maintainers usually reply within 1 day
Nobody has claimed this yet.
- #20065 by @brandonstensby-dotcom — closed without merging
Assessment
- Difficulty
- 2/5
- Estimated time
- 1-3 hours
- Newbie friendliness
- 76/100
Research direction
Start in backend/database/sync_recording_lineage.py and trace how lineage query limits and recording origin IDs are handled. Add hermetic unit tests for the requested limit bounds and whitespace validation, then run the relevant sync-recording-lineage tests to confirm the behavior is covered.
Written by the indexing model from the issue text.
Description
Summary
In backend/database/sync_recording_lineage.py, lineage query limits are unclamped and origin identifiers lack whitespace validation.
Proposed Fix
- Clamp query limits between 1 and 500.
- Validate recording origin IDs.
- Hermetic unit tests.
- Dominant language
- Python
- Stars
- 13.8k
- Forks
- 2.6k
- Avg merge
- 5h 40m
- Merged PRs (30d)
- 626
Getting set up
- No Dockerfile or Docker Compose file
- Has a pull request template
- Read the contributing guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
More from BasedHardware/omi
-
Python CLI: corrupt compressed responses hide an unknown write outcomePossibly taken @pranav440 claimed this 6 days ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
BasedHardware/omi#21295 · 1 comment ·
Maintainers usually reply within 1 day
-
chore(gateway): emit INFO budget reserved/settled logs for proactivity v2 (chip task_2855f4ec)Possibly taken @india3Eth claimed this 10 days ago. Openbackend
Difficulty 2/5 1-3 hours Newbie friendliness 68/100
BasedHardware/omi#20940 · 1 comment ·
Maintainers usually reply within 1 day
-
Ray-Ban Meta: a failed audio start is swallowed and capture shows recording with no audioPossibly taken A pull request linked to this issue is open or already merged. Openbug capture flutter p1
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
BasedHardware/omi#20782 · 1 comment ·
Maintainers usually reply within 1 day
-
[Bug]: Memory non-active routes lack blank string validation and timezone normalizationPossibly taken @brandonstensby-dotcom claimed this 17 days ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
BasedHardware/omi#20084 ·
Maintainers usually reply within 1 day
-
[Bug]: Memory apply store lacks identifier validation and deletion receipt limit clampingPossibly taken @brandonstensby-dotcom claimed this 17 days ago. Open
Difficulty 2/5 1-3 hours Newbie friendliness 78/100
BasedHardware/omi#20075 ·
Maintainers usually reply within 1 day
All issues in BasedHardware/omi
Similar issues
-
documentation
Difficulty 2/5 1-3 hours Newbie friendliness 72/100
ianktoo/project-monitor#14 ·
-
security
Difficulty 2/5 1-3 hours Newbie friendliness 84/100
btclib-org/btclib-wallet#478 ·
Maintainers usually reply within 1 day
-
security
Difficulty 2/5 1-3 hours Newbie friendliness 75/100
btclib-org/btclib#2727 ·
Maintainers usually reply within 1 day
-
good first issue lead variant:container
Difficulty 2/5 1-3 hours Newbie friendliness 76/100
radialmonster/orf-cleanroom#10 ·
Maintainers usually reply within 1 day
-
gdsa opencode: keep-alive cleanup trap fails with 'keepalive_pid: unbound variable'Possibly taken @darribas claimed this today. Open
Difficulty 2/5 1-3 hours Newbie friendliness 85/100