Repository navigation
v1.4.1: IEC104 standard frame format revert, sqlalchemy[asyncio], privileged-port precheck - #26
Merged
Merged
Conversation
added 3 commits
September 30, 2026 20:57
…-byte ASDU header ConvergeLoop commit 1df6f50 mistakenly applied IEC 60870-5-101 (FT1.2) frame layout to IEC 60870-5-104: 2-byte APDU length (L1 L2) and 2-byte COT/OA in the ASDU header. Standard 104 frames are 68 L(1) C1..C4 with a 6-byte ASDU header (TI+VSQ+COT(1)+OA(1)+CA(2 LE)). The regression made the server drop standard masters on connect (0x07 control byte rejected as L2) and reintroduced the CA/IOA x256 bug fixed in 4f7f803, which shipped in v1.4.0. Restore the wire-verified 4f7f803 server.
…ASCII alembic.ini - pyproject: declare sqlalchemy[asyncio]>=2.0.0. SQLAlchemy 2.1 dropped the default platform greenlet dependency (extra-only); alembic resolved to 2.1.1 in the v1.4.0 image so containers crashed at migration step with 'No module named greenlet' - alembic.ini: ASCII-only comments - configparser decodes it with the system locale, which crashes on Windows GBK systems when non-ASCII characters are present (also affected local pip installs) - engine.start_protocol: pre-flight bind probe for ports < 1024. PermissionError used to be misread as 'port in use', auto-switch then walked the whole privileged range and died with 'No free port found' (API 503) - the reported S7-on-102 failure in Docker. Now surfaces an actionable bilingual error (cap-add NET_BIND_SERVICE or port >= 1024) - defaults.get_friendly_error: map 'no free port found' to the same root-cause guidance (zh/en) - DEPLOYMENT.md + docker-compose.simple.yml: document/enable NET_BIND_SERVICE for privileged ports (S7 102) - tests/test_privileged_port_precheck.py: 4 regression cases
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.
v1.4.1 patch release (user-reported v1.4.0 issues):
Regressions: test_iec104_wire 7/7 green, test_privileged_port_precheck 4/4 green.