Skip to content

Conversation

@kentbull
Copy link
Collaborator

@kentbull kentbull commented Sep 8, 2025

This does a few things with the most important being upgrading to GLEIF-IT/keripy 1.2.8 including the exchange message processing fix

Other parts:

  1. Cleaned up imports
  2. Cleaned up logging with consistent root level module ogler usage and pulled in the TruncatedFormatter from vLEI-Server and KERIA.
  3. Refactored server setup to return a list of Doers so using it from an end-to-end system simulation test in SignifyPy is easy.
  4. Added a config bootstrapper so that sally server start ... is all you have to do, like the following command:
sally server start \
  --direct \
  --http 9723 \
  --salt 0AD45YWdzWSwNREuAoitH_CC \
  --name sally \
  --alias sally \
  --config-dir scripts \
  --config-file sally.json \
  --incept-file sally-incept.json \
  --passcode VVmRdBTe5YCyLMmYRqTAi \
  --web-hook http://127.0.0.1:9923 \
  --auth EMCRBKH4Kvj03xbEVzKmOIrg0sosqHUF9VG2vzT9ybzv \
  --loglevel INFO

This was caused by an indentation bug written by me back in Q1 2025 and committed to WebOfTrust/keripy in March 2025.
This generally clarifies logger usage to use an explicit name to get a logger so that the default hio logger is not used and so that all logs explicitly go to the "sally" labeled logger for traceability and clarity.
Returning a list of Doers from serving.setup makes Sally usable from an end-to-end simulation test, specifically from SignifyPy. This is how I diagnosed the double present bug.

And the config bootstrapper allows Sally to be started without a preexisting AID as the bootstrapper will OOBI with the configured witnesses prior to starting up. This does not matter as much any more now that direct mode is used since we tend to avoid using witnesses when we use direct mode. The config bootstrapper is useful only when the auto-incepted AID relies on a witness for inception on first startup.
@kentbull kentbull force-pushed the first-time-grant-failure branch from f179f99 to 0143f13 Compare September 8, 2025 15:43
@kentbull kentbull merged commit bc1b34f into GLEIF-IT:main Sep 8, 2025
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant