[tpm2] uriparser dependency

[tpm2] uriparser dependency

[Tricca, Philip B]

[-- Attachment #1: Type: text/plain, Size: 865 bytes --]

Hey all,

I've had a few people ask me about the liburiparser dependency that's been added to the mssim TCTI recently. Turns out this new dependency is less than convenient as some distros don't have multi arch packages for it. This library is only used to parse the configuration string at initialization time and replacing it with a simple key / value pair that we parse manually would be a trivial way to break the dependency. We are however very close to making a major release and making "significant" changes this close to the release is generally bad form. So if no one objects I'd like to hack up a patch that will replace our use of this library with a bit of code to parse key / value pairs and change the format of the configuration string for the mssim TCTI to something like "host=XX.YY.ZZ.AA,port=12345".

Let me know what you all think,
Philip

[-- Attachment #2: attachment.html --]
[-- Type: text/html, Size: 2702 bytes --]

Re: [tpm2] uriparser dependency

[Anderson, Daniel]

[-- Attachment #1: Type: text/plain, Size: 367 bytes --]

Please don't try and parse the host part or restrict the host to IPv4 addresses. This will allow hostnames and IPv6.
In other words, just pass everything after the host= part.  Also consider making port optional with a default (12345?).

This would allow things such as:
host=10.11.12.13
host=foo.bar.com,port=12345
host=2001:470:7b:5e7::2,port=12345

Dan