fix: remove unnecessary not null from queue table

Author: steve-chavez

README.md

@@ -64,8 +64,8 @@ The extension introduces a new `net` schema, which contains two unlogged tables,
             id bigint NOT NULL DEFAULT nextval('net.http_request_queue_id_seq'::regclass),
             method text NOT NULL,
             url text NOT NULL,
-            headers jsonb NOT NULL,
-            body bytea NULL,
+            headers jsonb,
+            body bytea,
             timeout_milliseconds integer NOT NULL
         )
     ```

sql/pg_net.sql

@@ -13,7 +13,7 @@ create unlogged table net.http_request_queue(
     id bigserial,
     method net.http_method not null,
     url text not null,
-    headers jsonb not null,
+    headers jsonb,
     body bytea,
     timeout_milliseconds int not null
 );

test/test_http_get_collect.py

@@ -174,3 +174,27 @@ def test_http_get_ipv6(sess):
 
     assert response is not None
     assert "Hello ipv6 only" in response[2]
+
+
+def test_http_get_null_headers(sess):
+    """net.http_get can have null headers"""
+
+    (request_id,) = sess.execute(text(
+        """
+        select net.http_get('http://localhost:8080', null::jsonb, null::jsonb, 100);
+    """
+    )).fetchone()
+
+    sess.commit()
+
+    response = sess.execute(
+        text(
+            """
+        select * from net._http_collect_response(:request_id, async:=false);
+    """
+        ),
+        {"request_id": request_id},
+    ).fetchone()
+
+    assert response is not None
+    assert "Hello world" in response[2]