Chore/cleanup - #2685
Merged
Merged
Conversation
…value that's no longer in the range.
chore: removed now replaced modules socketcan, virtualcan refactor: Made the OPC-UA server build inside Docker chore: Removed most of the obsolete code from the pcap-replay modules chore: Removed the plc-simulator
sruehl
approved these changes
Aug 11, 2026
spnettec
added a commit
to spnettec/plc4x
that referenced
this pull request
Aug 14, 2026
Upstream removed the test-only Milo dependency (chore/cleanup apache#2685) and reimplemented struct support on the self-built SecureChannel stack. This makes the earlier YOFC revert of upstream opcua files to Milo 1.1.4 compatible versions (934e171) obsolete; the merge left a hybrid that did not compile (upstream OpcuaConnection vs. reverted SecureChannel, OpcuaTag and generated readwrite classes). - Restore upstream versions of all opcua main/test/generated sources - Restore RawBinaryExtensionObjectWithMask.java deleted by 934e171 - Delete dead YOFC OpcuaPlcValueHandler: NULL-typed tags are now handled in OpcuaConnection (skips type override for PlcValueType.NULL) and the spi-values DefaultPlcValueHandler fix
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
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.
Please let me merge this.