Running dmr-client 0.011, with ResolverClient 0.0.9
Change the default repo to PROD
The CLI should report the CLI version, if we plan to have different RelsoverClient versions (that I would not recommend), report the full dependency chain versions
When running show --dtmi dtmi:com:example:TemperatureController;1 (without quotes), I got a nasty exception. Provide a clear error message
Output. Beyond the ILogger output I'd like to have a default output (that could be avoided with --no-logo), eg:
dmr-client [0.0.12/Parser 2.13.15] on netcoreapp 3.1.7/Ubuntu 20.04.1 LTS
Microsoft IoT Plug and Play Device Model Repository CLI
Executing: show --dtmi dtmi:a:b
Produce a clear result message without any Log filter
Review LogWarning and LogTrace outputs. I dont see a ton of trace messages, I'd move some of the current LogInfos to LogTrace. Consider having the default in Warning. Output expception details as LogTrace like 404, or not found host.
Manage JsonReaderExceptions like
Attempting to retrieve model content from 'https://aka.ms/dmrfaq/dtmi/meshsystems/txs/sensorsync-1.json'.
Unhandled exception: System.Text.Json.JsonReaderException: '<' is an invalid start of a value. LineNumber: 3 | BytePositionInLine: 0.
at System.Text.Json.ThrowHelper.ThrowJsonReaderException(Utf8JsonReader& json, ExceptionResource resource, Byte nextByte, ReadOnlySpan`1 bytes)
at System.Text.Json.Utf8JsonReader.ConsumeValue(Byte marker)
at System.Text.Json.Utf8JsonReader.ReadFirstToken(Byte first)
at System.Text.Json.Utf8JsonReader.ReadSingleSegment()
at System.Text.Json.Utf8JsonReader.Read()
at System.Text.Json.JsonDocument.Parse(ReadOnlySpan`1 utf8JsonSpan, Utf8JsonReader reader, MetadataDb& database, StackRowStack& stack)
Consider reporting Http Response Time in Trace mode
Manage not found local folders, eg:
Using repository location: ./ba
Unhandled exception: System.UriFormatException: Invalid URI: The format of the URI could not be determined.
at System.Uri.CreateThis(String uri, Boolean dontEscape, UriKind uriKind)
at System.Uri..ctor(String uriString)
at Azure.DigitalTwins.Resolver.ResolverClient.FromRemoteRepository(String repositoryUri, ILogger logger, ResolutionSettings settings) in /home/rido/code/iot-plugandplay-models-tools/resolvers/dotnet/Azure.DigitalTwins.Resolver/ResolverClient.cs:line 15
at Azure.DigitalTwins.Resolver.CLI.Program.InitializeClient(String repository, ILogger logger) in /home/rido/code/iot-plugandplay-models-tools/resolvers/dotnet/Azure.DigitalTwins.Resolver.CLI/Program.cs:line 63
at Azure.DigitalTwins.Resolver.CLI.Program.<>c.<<BuildResolveCommand>b__7_0>d.MoveNext() in /home/rido/code/iot-plugandplay-models-tools/resolvers/dotnet/Azure.DigitalTwins.Resolver.CLI/Program.cs:line 141
Running dmr-client 0.011, with ResolverClient 0.0.9
Change the default repo to PROD
The CLI should report the CLI version, if we plan to have different RelsoverClient versions (that I would not recommend), report the full dependency chain versions
When running show --dtmi dtmi:com:example:TemperatureController;1 (without quotes), I got a nasty exception. Provide a clear error message
Output. Beyond the
ILogger
output I'd like to have a default output (that could be avoided with--no-logo
), eg:Produce a clear result message without any Log filter
Review
LogWarning
andLogTrace
outputs. I dont see a ton of trace messages, I'd move some of the current LogInfos to LogTrace. Consider having the default in Warning. Output expception details as LogTrace like 404, or not found host.Manage
JsonReaderExceptions
likeConsider reporting Http Response Time in Trace mode
Manage not found local folders, eg: