Open
Conversation
Signed-off-by: fossabot <badges@fossa.io>
Author
|
Your license scan is passing -- congrats! Your badge status is now updated and ready to merge: |
MMaiero
pushed a commit
that referenced
this pull request
Oct 20, 2025
* feat: add NM introspection classes * build(deps): add dbus-java library * feat: more robust NetworkProperties * feat: add NM sources * docs: fix incorrect comment * fix: passphrase property conversion * ci: add auto-generated resources to Sonar exclusions * feat: add NMStatusServiceImpl * feat: NMDbusConnector instantiation and usage in NMConfigurationServiceImpl * refactor: remove filtering from getInterfaces() * feat: reintroduce NMDeviceType-based filtering for network status * feat: complete implementation for NMStatusServiceImpl * feat: add getInterfaceStatus method to DbusConnector * feat: add NMStatusConverter (WIP) * refactor: move NMStatusConverter inside nm.status * feat: correct mac address conversion * fix: buildEthernetStatus signature * tests: unit tests for NMDeviceState and NMDeviceTypeTest (#2) * feat: get IPAddress from interface * feat: hardcoded netmask for testing WIP * fix: setBroadcast -> setNetmask * fix: NPE on NetMask * feat: correctly populate Netmask from Prefix * feat: retrieve all IPs from interface * refactor: remove unnecessary parameters * feat: detect loopback interface type * feat: add support for Loopback interface status * fix: set loopback as virtual interface * refactor: DRY NMStatusConverter * refactor: add log entry for failed initialization * refactor: proper log entry for failed configuration apply * refactor: run KuraCleanup_v3 profile * feat: handle case where device is not connected while retrieving status * fix: wrong merge conflict resolution * feat(wifi): add support for `ignoreSSID` property * refactor: split setIP4Status into two smaller methods * style: add copyright notice to auto-generated files * style: add missing copyright headers * tests: NetworkProperties test coverage (#3) * refactor: group common code inside `org.eclipse.kura.nm.common` package * refactor: rename KuraInterfaceStatus -> KuraIpStatus * refactor: rename fromKuraInterfaceStatus -> fromKuraIpStatus * refactor: rename NMDeviceEnable -> KuraInterfaceStatus * style: fix copyright header * refactor: rename org.eclipse.kura.nm.common -> org.eclipse.kura.nm * refactor: move autogenerated sources in `/src/main/java/org/freedesktop` * test: use consistent naming convention * style: applied Kura style and clean up * test: fix wrong `get` method tests * test: fix asserts * test: add expectation for easier debugging * test: add more expectations for easier debugging * feat: stricter control on NetworkProperties access * feat: more robust error handling for NMConfigurationServiceImpl * fix: add forgotten Optional.get() * fix: pairwise and group conversion * build: remove unnecessary dependency * build: download dependency at build time * chore: add downloaded libs to .gitignore * build: hopefully fix test build * feat: add dependency injection for NMConfigurationServiceImpl * test: fix NMConfigurationServiceImplTest tests * refactor: use switch statements instead of maps, better error handling * ci: fix wildcard pattern for Sonar scan * refactor: use switch statement instead of map, better error handling * style: fix pom.xml formatting * style: fix pom formatting * refactor: replace string literals with constants for DBus properties * refactor: remove unnecessary replaceAll * refactor: use primitive boolean expressions * refactor: add manageConfiguredInterface method to reduce cognitive complexity * fix: wrong visibility modifier * refactor: remove expression always evaluating to true * fix: forgotten "s" in string format * refactor: add connection and method constants --------- Co-authored-by: G_Ivo <gregorybivo@gmail.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 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.
Your FOSSA integration was successful! Attached in this PR is a badge and license report to track scan status in your README.
Below are docs for integrating FOSSA license checks into your CI: