Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Serval improvements #91

Merged
merged 7 commits into from Feb 25, 2014
Merged

Serval improvements #91

merged 7 commits into from Feb 25, 2014

Conversation

dismantl
Copy link

overhauled serval-dna plugin crypto code. the only visible change for the user is that the signing and verifying commands give more output, including keys.

to test, make sure app advertising works, and that olsrd-mdp works.

includes https://github.com/opentechinstitute/luci-commotion-apps/pull/68

@jheretic
Copy link
Member

@dismantl, with this pull request added, olsr-mdp works as expected. However, there is a problem with application advertisements. Only local advertisements show up, and when running commotion-service-manager manually it gives the following relevant output:

[LOG_INFO] (commotion-service-manager.c:522) Browser: NEW: service 'fb2064837f8946ba4fdaf0f60f4fbefae632c72f' of type '_commotion._tcp' in domain 'mesh.local'
[LOG_INFO] (commotion-service-manager.c:522) Browser: NEW: service 'fb2064837f8946ba4fdaf0f60f4fbefae632c72f' of type '_commotion._tcp' in domain 'mesh.local'
[LOG_WARNING] (commotion-service-manager.c:439: errno: None) (Resolver) Invalid signature: fb2064837f8946ba4fdaf0f60f4fbefae632c72f -> A
[LOG_INFO] (commotion-service-manager.c:129) Removing service announcement: fb2064837f8946ba4fdaf0f60f4fbefae632c72f
[LOG_INFO] (uci-utils.c:332) Application is local

Apparently, even local advertisements are not actually being validated. Remote advertisements give the same output (with the exception of the 'Application is local' part). Running commotiond from the commandline in parallel gives no particular output, but it was not compiled with debugging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants