-
Notifications
You must be signed in to change notification settings - Fork 932
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
Permit multiple attempts to load PostgreSQL CDR module. #737
Permit multiple attempts to load PostgreSQL CDR module. #737
Conversation
cherry-pick-to: 21 |
The UserNote in the commit message isn’t necessary |
df183e7
to
fdd330d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Besides the specific comments I made, we use tabs for indentation, not spaces.
fdd330d
to
1397494
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Completely up to you, but I believe this results in the desired behavior and is slightly less verbose.
1397494
to
0941837
Compare
0941837
to
4bbfb9d
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The commit message first line does not comply with the standard:
Missing or corrupt cdr_pgsql.conf configuration file can cause the second attempt to load the PostgreSQL CDR module to crash Asterisk via the Command Line Interface because a null CLI command is registered on the first failed attempt to load the module. Resolves: asterisk#736
f789b77
into
asterisk:master
Successfully merged to branch master and cherry-picked to ["21","20","18"] |
Missing or corrupt cdr_pgsql.conf configuration file can cause the second attempt to load the PostgreSQL CDR module to crash Asterisk via the Command Line Interface because a null CLI command is registered on the first failed attempt to load the module.
Resolves: #736