search

HealthCatalyst / Fabric.Identity

Identity service to provide authentication
Apache License 2.0
12 stars 6 forks source link

removed validation from install-discovery script #351

Closed hc-jordenlowe closed 5 years ago

hc-jordenlowe commented 5 years ago

Weird bug. If you call this individually, then it validates the hashtable.

If you call it from the main script, it works with no validation.

hc-jordenlowe commented 5 years ago

Looks good - the issue is that you're validating a string with your decorators but $_ refers to a hashtable which doesn't work with TestPath ($ refers to $configStore, not $configStore.Path). If you want to validate path, you need to use $_.Path

neat, i can add that in then.

hc-jordenlowe commented 5 years ago

looks like there is an issue with hash tables. Instead of the validation being done in the parameter in an aspect oriented fashion, i will just add the code to the first lines of the function.