Add installable OpenWrt package - #1
Open
safesploit wants to merge 5 commits into
Open
Conversation
Implement Phase N (Installable OpenWrt Edition) with: - Package infrastructure: Makefile, staging script, UCI configuration - Read-only rpcd API (firewall.visualiser) with six methods via ucode service - Frontend live mode with authenticated router access and bounded polling - Platform abstraction layer supporting offline and openwrt runtime modes - OpenWrt API client with automatic session handling and normalization - Dedicated listener setup command for management-address-specific HTTPS - SDK build validation and optional x86_64 QEMU smoke test in GitHub Actions - Comprehensive contract tests covering API, ACL, package layout, and shell scripts - Updated documentation and README with package installation instructions
Split the complex single-line SSH command into separate steps to improve readability. The change separates the file existence check from the UCI value verification, making the test easier to understand and debug.
safesploit
changed the title
OpenWrt feed names cannot contain hyphens. Changed FEEDNAME from 'firewall-visualiser' to 'firewall_visualiser' to comply with OpenWrt naming requirements. Added a test to enforce this constraint going forward.
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
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.
Implement Phase N (Installable OpenWrt Edition) with:
Summary
Describe what this pull request changes.
Type of Change
Validation
Security Considerations
Describe any security impact, assumptions or risks.