Andrew Whitehead

Implement transaction options API

2

This adds `begin_with` methods for creating transactions with additional options. The exact options object varies depending upon the backend. For MS-SQL and AnyDatabase the options are currently just `()` although...

Asynctest replacement

1

The `asynctest` library which we use extensively in unit tests is no longer being maintained, and does not function properly under Python 3.8+. Coincidentally, Python 3.8 with mock 4.0 introduces...

Use contextvars for managing dependency injection

2

If we increase the minimum required Python version to 3.7, then we can support the use of [contextvars](https://docs.python.org/3/library/contextvars.html). I'm not sure what an implementation would look like yet, but this...

Update KeyGen procedure to use hash_to_scalar

2

Since we don't use HKDF anywhere else and aren't trying to be compatible with the BLS signatures spec (different salt). It's a bit odd using camel case for the input...

Support for HSMs

1

I'm just opening this issue for discussion of whether the current algorithms preclude support for signing by an HSM (hardware security module) or other types of key vaults. Some questions...

Simpler SpkGen option

21

I'm curious what's wrong with this simpler SPK alternative which doesn't specifically use `h_0` and `s`, treating them as a hidden message: https://hackmd.io/8t8o56mvTL6p-VZyayLnTA?view One benefit here is that blind signatures...

Keygen questions

2

The [current spec](https://mattrglobal.github.io/bbs-signatures-spec/#name-keygen) defines the keygen salt as `BBS-SIG-KEYGEN-SALT-`, while this library uses the value `BLS-SIG-KEYGEN-SALT-`. The second is consistent with the BLS signatures spec [draft 2](https://datatracker.ietf.org/doc/html/draft-irtf-cfrg-bls-signature-02#section-2.3), and the method...

elliptic-curve: CofactorGroup requirement for hash2curve implementation

3

I was looking at implementing the hash2curve traits for bls12-381 (which has an implementation, but not using these traits currently), but ran into an issue with the CofactorGroup requirement on...