Luca Ferrari

The function `as_bytes` accepts suffixes like 'M' for megabytes. It would be nice and more natural to allow the user to specify also 'MB', that means allow for a double...

[#301] Fix mispelled default pid file.

2

Fixes the name of the default pid file. Close #301

Improve pid file management

4

These commits improve a little the management of the pid file by `main` application.

The pid file is mispelled:

While working on #298 and re-reading the documentation of `open(2)`, I found that `open(2)` fails when `O_CREAT | O_EXCL` are specified as flags if the file is already there. Therefore...

In the `main` there are some exit conditions, e.g., due to not being able to bind a socket, that perform an `exit(1)` without removing the pid file (e.g., ). All...

While working on #296 I saw that `configuration.c` has a few functions that have error paths like : ``` error: free(master_key); free(password); free(decoded); if (file) { fclose(file); } return 1;...

Better checks for `pgagroal.conf`

2

This happens on current HEAD 1296cc4216c73119a1ff4c3a3ffd0c610ca04f69; assume the following dumb configuration file `pgagroal.conf`: ``` $ cat /etc/pgagroal/pgagroal.conf [pgagroal] host = * port = 5432 unix_socket_dir = /tmp management=6432 log_type =...

Refactor `pgagroal-admin` commands

1

Related to #253 and discussion in #289 (see ): `pgagroal-admin` could have a more consistent set of commands grouped by *command* and *subcommand*. An example could be: `user` and then...

[#253] Refactor commands in `pgagroal-cli`

7

Now `pgagroal-cli` has a set of "logically" grouped commands and subcommands. For example, all the commands related to shutting down the pooler are under the `shutdown` command, that can operate...