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

Add trust_input option to postgresql_slot module #298

Conversation

andytom
Copy link
Contributor

@andytom andytom commented May 7, 2020

SUMMARY

Have added a trust_input option to the postgresql_slot module. This
only checks the session_role since all other options are passed as
parameters.

Related to #106 and #210.

ISSUE TYPE
  • Feature Pull Request
COMPONENT NAME

plugins/modules/database/postgresql/postgresql_slot.py

ADDITIONAL INFORMATION

@ansibullbot ansibullbot added WIP feature This issue/PR relates to a feature request integration tests/integration module module tests tests labels May 7, 2020
Have added a trust_input option to the postgresql_slot module. This
only checks the session_role since all other options are passed as
parameters.
@andytom andytom force-pushed the feat/add_trust_input_postgresql_slot branch from dd51d65 to ff75a2d Compare May 7, 2020 10:31
@andytom andytom marked this pull request as ready for review May 7, 2020 11:07
@andytom andytom changed the title [WIP] Add trust_input option to postgresql_slot module Add trust_input option to postgresql_slot module May 7, 2020
@andytom
Copy link
Contributor Author

andytom commented May 7, 2020

ready_for_review

Copy link
Contributor

@Andersson007 Andersson007 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
shipit

@Andersson007 Andersson007 merged commit f887aff into ansible-collections:master May 7, 2020
@Andersson007
Copy link
Contributor

@andytom thanks for the contribution!
merged into master

@andytom andytom deleted the feat/add_trust_input_postgresql_slot branch May 7, 2020 15:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature This issue/PR relates to a feature request integration tests/integration module module tests tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants