Remove STRICT from all functions

[steve-chavez]

Problem

While working on supporting DELETE with a request body, I added a new parameter with a NULL default to grant backwards compat:

create or replace function net.http_delete(
...
    body jsonb default NULL
)
    returns bigint
    strict
    volatile
    parallel safe
    language plpgsql

Then while doing:

select net.http_delete('http://localhost:8080/delete');

I kept getting a NULL result, after a lot of time of head-scratching, I realized it was because of the STRICT modifier in the function. That clause is always surprising, the behavior escapes the natural flow of code.

It also causes performance problems (PostgREST/postgrest-docs#224). IMO it should NEVER be used.

Solution

Remove STRICT from all functions.

While at it, clean up the signatures of the functions from unnecessary words like volatile, etc.