Skip to content
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

return all values from Get-TppAttribute, including default policy #2

Closed
Curtmcgirt opened this issue Apr 6, 2021 · 2 comments · Fixed by #45
Closed

return all values from Get-TppAttribute, including default policy #2

Curtmcgirt opened this issue Apr 6, 2021 · 2 comments · Fixed by #45
Assignees
Labels
enhancement New feature or request

Comments

@Curtmcgirt
Copy link

Environment

Operating System:
VenafiTppPS version:
PowerShell version:

Steps to reproduce

i have found that if i do get-tppattribute on a known-good, IIS-binding-updating, app created in the GUI with the IIS binding port set to 443, there is no "binding port" in the results of get-tppattribute. there is only the "port", which is the winRM port 5985/6.

Expected behavior

Actual behavior

Screenshots

@gdbarron
Copy link
Collaborator

gdbarron commented Apr 6, 2021

The config/readall API only returns those values that are different than default policy settings. You can also see the same value list on the Support/Attributes tab for the app you created. The description for the function could definitely make this a bit clearer.

To get all values, I suspect we'd need to query for all attribute names for a given class and then request those values, but not 100% sure on that.

@gdbarron gdbarron changed the title binding port missing from get-tppattribute return all values from Get-TppAttribute, including default policy Apr 14, 2021
@gdbarron gdbarron transferred this issue from gdbarron/VenafiTppPS May 6, 2021
@gdbarron gdbarron self-assigned this Nov 27, 2021
@gdbarron gdbarron added the enhancement New feature or request label Nov 27, 2021
@gdbarron gdbarron linked a pull request Nov 27, 2021 that will close this issue
@gdbarron
Copy link
Collaborator

@Curtmcgirt this was added in 3.4.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants