feat: improve config debug-ability

[0x009922]

Description

This PR contains a major improvement of the work done in #4239. This is a big step towards better debug-ability of Iroha configuration system due to better errors. And yet, there are still many things to improve further!

Key points:

• New ConfigReader API, which handles configuration sources in a dynamic manner rather than relying on serde completely, thus giving high flexibility in how to handle different cases. It is also built targeting Iroha domain specifically and doesn't have any extra stuff. Much less boilerplate.
• Heavy use of error_stack crate, which gives higher control over errors reporting than eyre does.
• --trace-config works! Uses log & stderrlog crates for it. Seems to be not a heavy solution for this. It is completely separate from tracing stack.

Further enhancements

There are still many things that might be improved.

• Enhance use of attachments API provided by error_stack:
  • Use hooks to colour them conditionally
  • Attach hints and suggestions
  • Unify attachments further as specific structs, not just format!
• Use spans to config files, and build reports showing the actual file contents
• Fine-tune API of ConfigReader, use better naming and abstractions, perhaps.
• Optimise internals of ConfigReader. There is lots of naive code and extra allocations.
• Find a way to reduce extra context changes in error_stack. Sometimes it would be really cool to just "forward" underlying context without changing it, but currently it's how error_stack is designed. So, we have sometimes stacks like "Failed to load config -> Failed to load config -> failed to read file system -> ..."
• Possibly, introduce a slim compile feature for Iroha, which will remove --trace-config support alongside with log & stderrlog dependencies, and more
• Use a single multihash string for private keys, thus removing necessity to set private keys in env as two separate variables (e.g. PRIVATE_KEY_ALGORITHM + PRIVATE_KEY_PAYLOAD, and removing cratchy env_custom API from ConfigReader
  • [suggestion] Adopt multihash format for private keys #4412
• Migrate to Axum, finally! Errors handling in Torii might be improved a lot.
  • [refactor] Migrate from warp to axum/actix #3776

Linked isues

Closes #3470, #4299, #4300

Gallery

Note: I've changed some bits after making these screenshots.

Some terminal screenshots

TODO

• Document internals of iroha_config_base
• Fix iroha_client_cli compilation failure
• Enabled failing test back: fails_with_no_trusted... in cli/src/lib.rs
• Self-review

[github-actions]

@BAStos525

[0x009922]

@DCNick3, I would appreciate if you can take a look at the procmacro I wrote here, in config/base/derive.

[0x009922]

Debugging CI: new reports are really helpful!

[DCNick3]

A bit concerned with the inclusion of source code location in the error trace (the at some/source/location.rs lines). I don't think it's useful for users and questionably useful for iroha developers. Maybe remove it/gate being some verbosity flag?

[0x009922]

A bit concerned with the inclusion of source code location in the error trace (the at some/source/location.rs lines). I don't think it's useful for users and questionably useful for iroha developers. Maybe remove it/gate being some verbosity flag?

I am concerned too. There are also complete backtraces attached, which are displayed in dev environment. Anyway, we can disable Location's printing using the hooks. I used it in tests to capture snapshots. Will think about it.