Werner Koch wrote:
Hi,

find attach the patch which makes all 3 test cases work with Ed25519.
There are some minor hacks in the test cases to allow enabling of
Libgcrypt debugging and also some minor output style changes.

There is one FIXME in the code:

  /* FIXME: mpi_print creates an unsigned integer - is that intended
     or should we convert it to a signed integer (2-compl)?  */
  mpi_print (xbuf, sizeof (xbuf), result_x);

X may be positive or negative but GCRYMPI_FMT_USG ignores the sign.
Thus this is not what we actually want.  Should we change it to 2-comp
(GCRYMPI_FMT_STD) so that we have a proper value?  Given that the curve
is 255 bit this should alwas fit int the 256 bit buffer.  Another option
would be to use the EdDSA method for the sign but that is optimized to
easily recover x and would be more work.  Or we store the sign in the
high bit.  t all depends on what you want to write into the protocol
specs.

I would also like to revert the way we distinguish between Ed25519 with
and without ECDSA:  The way we do it right now is by assuming the
Ed25519 is always used with EdDSA unless a flag has been set.  This is a
bit surprising and requiring the "(flags eddsa)" would be a less
surprising interface.

Salam-Shalom,

   Werner

- corrected discount for continuous smdp

internal time logging

Add doublequotes; also, r for 'revision'

- doc

-improving API design

-florian wants sane dot graphs

- header

- fixes, doc

- document, alpha renaming

Beautify the version script

- refactoring, extra checks

- cleanup

-handle PENDING

- change key exchange messages to own encapsulation

-do not set PENDING on shorten

fix tests and do not assert since this will break make check

skeleton for secretsharing

- implemented missing set functionality
- secretsharing api changes

block expiration

minor fixes + test for shadows record filtering and usage

implementation of shadow record functionality:
on decrypt iterate over included records
- if a shadow record is found, check if:
-- a non expired, non-shadow record for this record type exists: if yes: filter shadow record
-- shadow record is expired
if not:
- remove shadow flag and include record

- new message for mesh KX

new test to lookup shadow record

- fix: all records were created as shadow record by default
- new:
-- printing record flags
-- improved error message on deletion

fix for lookup

doxygen

separated threshold from other crypto parameters

typo

typedef for decryption result callback

clarifications

fixed include guard

- secretsharing api proposal

logread

benchmark with updates

update info for benchmark

adding additional INFO

-doxygen

centralized function to start solution

writing data files

- missing headers

-simplify utf8_tolower/upper APIs

- bulk lock
- epoch triggered after changes of environment

gnuplot basics

improved benchmarking

-getting rid of some warnings

-more fix

-fix gnurl checks

clean up unused vars

implemented solver information callbacks for benchmarking

basic benchmark functionality working

clean up

- doxygen

fix assertion: address has to be removed from peermap before notifying solver

improved message

- kx

minor fixes and log level changes

-man page for gnunet-conversation-test

- some of the missing set union functionality implemented

-match exact types from message struct

fix peer creation

function to fill buffer with random values

fix for quota loading

next steps to generic perf test

move definition

fixed

-indent

- use configuration template for duplicate configurations

- allow localhost testing

add confs to make dist

- doc link to hostfile format

- fixed assertion due to task not testing for shutdown as a reason

next step for generic solver test

initial changes for generic solver performance test

removing unused tests and clean up makefile

- fixes

- tolerate zero cache size

replace zone iteration with record lookup

replace iteration with lookup

- missing header

- cleanup; move to hashmap32

extended plugin api to support lookup function
added new index to both plugins
implemented lookup functionality in both plugins
namestore uses lookup function

- warn upon failed $-expansions

-better doxygen

-better comment

-log parse errors as WARNINGS

-improvements to parsers

-make proper use of new policy parser

-faster port range generation, O(n) instead of O(n^2)

-converting regular expressions of vpn/pt to non-binary format and adding proper policy parsing

-moving code to parse IPv4/IPv6 subnet specifications to strings and making it part of the public API

-doxygen

-eliminate dead code

- encryption stuff

api communication done

added lookup test + fix in test

- clean protocol number
- added lookup protocol numbers
- implemented lookup sending

- fix connection destroy accounting

- remove debug

- fix double free