-
Notifications
You must be signed in to change notification settings - Fork 4
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Verify format of web_app_pool_info
output types
#14
Comments
cc @ronger4 |
I will investigate further and see if other properties differ in type. |
I think it’s still a timespan but I could be wrong but yes the input format of the attributes should align with output value from the _info module. |
I investigated further.
The reason that the output in the end is a dictionary formatted timespan and not string format is most probably due to the fact If we want to align it to be a string too, I can convert it manually to string during the handling in the module itself.
The advantage of the way of how it is handled now is that the output is more readable with the dictionary representation. |
SUMMARY
The return values of the
web_app_pool_info
attributes should be in the same format that we accept forweb_app_pool
. For example therecycling.periodicRestart.time
value is returned as a serializedTimeSpan
object which is a dictionary with various properties but theweb_app_pool
module requires the input format like00:00:05:00.000000
. We should make sure that known types likeDateTime
as converted to the same format we expect. I've not dug too deeply into it but theTimeSpan
values are the first ones that jump out, we should check other types are returned in a similar fashion.ISSUE TYPE
COMPONENT NAME
web_app_pool_info
The text was updated successfully, but these errors were encountered: