-
Notifications
You must be signed in to change notification settings - Fork 1
Develop #69
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
Merged
Merged
Develop #69
Changes from all commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
6a2aaed
ci: add CodeQL config to suppress IoT privacy false positives
markus-lassfolk b1ce6f8
feat: add GlitchTip error reporting (opt-out for beta)
markus-lassfolk 357ea14
merge: resolve conflict, use opt-out error reporting with public DSN
markus-lassfolk 6b6b7bf
fix(mqtt): run blocking paho connect in executor to avoid sync I/O onβ¦
markus-lassfolk f889ac1
feat(local): add publish_command() to YarboLocalClient
markus-lassfolk 5a33dc8
feat: add ~60 typed command methods, publish_command alias, destructiβ¦
markus-lassfolk b446eac
merge: resolve conflicts from main into develop
markus-lassfolk f0d773d
fix: address review feedback
markus-lassfolk 211e802
merge: incorporate main into develop (resolve conflicts)
markus-lassfolk ec0f6e3
fix: remove duplicate method definitions from merge
markus-lassfolk 98aceb3
style: ruff format error_reporting.py and local.py
markus-lassfolk b3a72d0
fix: restore _request_data_feedback, fix mypy errors
markus-lassfolk ba106ba
fix: restore all typed command methods in local.py
markus-lassfolk 479f00c
fix: add missing methods and constructor args for mypy compliance
markus-lassfolk 3818193
fix: add head validation, destructive safeguards, fix payloads and TLS
markus-lassfolk ce01b58
fix: tls_set_context and chute_steering_work kwarg
markus-lassfolk 80bfbf6
fix: TLS test assertions, sentry importorskip, ruff format
markus-lassfolk b931ef4
fix: ruff lint and format
markus-lassfolk d284139
fix: remove duplicate DSN, fix opt-out comment, publish_command delegβ¦
markus-lassfolk 84791ca
fix: TLS tests call _create_and_connect_paho directly (CI thread mockβ¦
markus-lassfolk 3daca57
fix: ruff format test_mqtt.py
markus-lassfolk File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,20 @@ | ||
| name: "Yarbo CodeQL Config" | ||
|
|
||
| # Disable queries that flag core IoT features as privacy issues | ||
| # GPS coordinates, IP addresses, MAC addresses, and hostnames are | ||
| # fundamental to robot mower discovery and control β not PII leaks. | ||
| query-filters: | ||
| - exclude: | ||
| tags contain: security/cwe/cwe-532 # Clear-text logging (we handle this ourselves) | ||
| - exclude: | ||
| tags contain: security/cwe/cwe-359 # Privacy violation (GPS, IP, MAC are core features) | ||
| - exclude: | ||
| tags contain: security/cwe/cwe-200 # Information exposure (device telemetry is the product) | ||
| - exclude: | ||
| id: py/clear-text-logging-sensitive-data | ||
| - exclude: | ||
| id: py/clear-text-storage-sensitive-data | ||
|
|
||
| paths-ignore: | ||
| - tests | ||
| - archive |
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,27 @@ | ||
| name: "CodeQL" | ||
|
|
||
| on: | ||
| push: | ||
| branches: [main, develop] | ||
| pull_request: | ||
| branches: [main, develop] | ||
|
|
||
| permissions: | ||
| security-events: write | ||
| contents: read | ||
|
|
||
| jobs: | ||
| analyze: | ||
| name: Analyze | ||
| runs-on: ubuntu-latest | ||
| strategy: | ||
| matrix: | ||
| language: [python] | ||
| steps: | ||
| - uses: actions/checkout@v4 | ||
| - uses: github/codeql-action/init@v3 | ||
| with: | ||
| languages: ${{ matrix.language }} | ||
| config-file: .github/codeql/codeql-config.yml | ||
| - uses: github/codeql-action/autobuild@v3 | ||
| - uses: github/codeql-action/analyze@v3 |
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
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.
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.
Uh oh!
There was an error while loading. Please reload this page.