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

Refs #31980 - improve Ansible as authorized feature #389

Closed

Conversation

ezr-ondrej
Copy link
Member

In 1b90f87 we allowed Ansible as
authorized feature on smart proxy, but we acutally overriden the Array
so Puppet wouldn't be authorized_feature.

Also adding a test that config report can be created with proxy only
having Ansible feature.

@domq
Copy link

domq commented Mar 3, 2021

Ah, my bad - Ther's still so much I don't know about Ruby! Thanks for the fix.

@ezr-ondrej
Copy link
Member Author

Ah, my bad - Ther's still so much I don't know about Ruby! Thanks for the fix.

No worries this is quite a strange ruby behaviour.
It's more of a my fault, I should have test it better before merging ;)

In 1b90f87 we allowed Ansible as
authorized feature on smart proxy, but we acutally overriden the Array
so Puppet wouldn't be authorized_feature.

Also adding a test that config report can be created with proxy only
having Ansible feature.
@ezr-ondrej
Copy link
Member Author

This has been fixed by #491, closing

@ezr-ondrej ezr-ondrej closed this Jan 12, 2022
@ezr-ondrej ezr-ondrej deleted the authorized_features_test branch January 12, 2022 14:50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants