[pull] master from vernesong:master#41
Open
pull[bot] wants to merge 3569 commits into
Open
Conversation
Problem:
- The check_mod() function uses Bash 4.0+ syntax ${1^^} to convert module names to uppercase
- OpenWrt uses busybox ash as /bin/sh, which doesn't support this syntax
- This causes "syntax error: bad substitution" and prevents module detection
- Affects systems where kernel modules are built-in (CONFIG_xxx=y)
Solution:
- Replace ${1^^} with POSIX-compatible: $(echo "$1" | tr a-z A-Z)
- Use a-z/A-Z instead of [:lower:]/[:upper:] (BusyBox tr doesn't support character classes)
Tested on:
- iStoreOS 22.03.6 with BusyBox v1.35.0 (ash)
- Successfully detects CONFIG_TUN=y in /proc/config.gz
Fixes module detection for built-in kernel modules on OpenWrt/BusyBox systems.
Co-authored-by: Patrick Gu <username@users.noreply.github.com>
- Remove default checked attribute from Running Mode radio button - Remove default checked attribute from Proxy Mode radio button - Align behavior with other setting components (Area Bypass, Sniffer, etc.) - Prevent showing incorrect state before data loads This ensures all radio button groups show no selection until actual configuration data is retrieved from backend. 🤖 Generated with [Claude Code](https://claude.com/claude-code) Co-authored-by: pgu001 <pgu001@users.noreply.github.com> Co-authored-by: Claude <noreply@anthropic.com>
* refactor(switch_dashboard): align buttons inline and simplify JS - Wrap three cbi-value-field divs in a flex container to display buttons in one row - Extract makeBtn() helper to reduce repeated innerHTML string building - Wrap per-panel code in IIFE to avoid variable naming conflicts - Register switch/delete/default functions once via window.xxx || function pattern - Move btn.disabled = false into XHR callback for correct async behavior * fix class name --------- Co-authored-by: vernesong <42875168+vernesong@users.noreply.github.com>
…orruption (#5136) * fix: add geo download validation (HTML detection + file size check) - Add HTML header sniffing to detect CDN error pages (e.g. CF 200 + HTML) - Add minimum file size validation (1MB for GeoIP/GeoSite, 10KB for ASN/Country.mmdb) - Keep all LOG_OUT in English per maintainer request - Add Chinese (zh-cn) and Spanish (es) .po translations for new messages * chore: adjust indentation --------- Co-authored-by: vernesong <42875168+vernesong@users.noreply.github.com>
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 subscribe to this conversation on GitHub.
Already have an account?
Sign in.
12 participants
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.
See Commits and Changes for more details.
Created by
pull[bot] (v2.0.0-alpha.4)
Can you help keep this open source service alive? 💖 Please sponsor : )