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

Disable postgres USER triggers #299

Merged
merged 1 commit into from
Aug 15, 2018
Merged

Disable postgres USER triggers #299

merged 1 commit into from
Aug 15, 2018

Conversation

gforcada
Copy link
Contributor

To be able to disable ALL triggers on a database, one needs a user that can disable system triggers.

Should fix #290

To be able to disable ALL triggers on a database, one needs a user that can disable system triggers.
@avelis
Copy link
Collaborator

avelis commented Aug 15, 2018

@gforcada I am not a PG user here. So in theory, I am ok with the change.

@avelis avelis merged commit d4b4403 into jazzband:master Aug 15, 2018
@gforcada gforcada deleted the patch-1 branch August 15, 2018 18:30
@devmonkey22
Copy link
Contributor

Thanks @gforcada, the patch from ALL to USER works for me too in PG.

@gforcada
Copy link
Contributor Author

@avelis could we get a release? TIA! 👍

@avelis
Copy link
Collaborator

avelis commented Aug 16, 2018

@gforcada Let me see what I can do. I appreciate the patience and all the contributions this library gets.

@mOscuro
Copy link

mOscuro commented Oct 22, 2018

Hi! any news about a possible release? I'm getting impacted by this as well. Thanks!

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

Successfully merging this pull request may close these issues.

silk_clear_request_log fails on Postgres
4 participants