agoo 2.14.3 → 2.15.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 843f3aa4a15a6ab57c7ff231930405c9ca55d3a6e33da3169a0677c420195fbe
-  data.tar.gz: 03f6b8760c6f08c0a3f9b5f75c747b77d1d4666b00af20d7db231418dfeb3c69
+  metadata.gz: ad246b7aa0e634eff81c6c9bdec4d3567115a3beb8b6c237e25f87215fc2dbe3
+  data.tar.gz: 03b6217fd351024938607dd036dcfd437a0dd967b22eca2b51aa0ed8a041d969
 SHA512:
-  metadata.gz: '0684900cea98fbe812e5ae973ccd15dfaa054218340238dedfde382c535340945aea9857b98366fe52b2ff865129b299af541e940112200f2191b3b2247dfd58'
-  data.tar.gz: 95e68a0a7bc5a3eb8cd34b9853e8d89285ef9a1c78765bf970e80cdeb51f7f41f53db4948ca2885e29eef3fa7a6b738e015d01c4cbb25e41b2f50bac1fcb50a9
+  metadata.gz: a244ea77e0f13688e2e6237c31eb320af4c0eeb9780f6023ef6337db7f64839ce543a27d5890155fb0908f0ce6567733b750be0d7b7c4de246430c54a736b6f4
+  data.tar.gz: abf7f274c15852b21b64ac6018b03264d08968902b9ad71d4f56d84a5ea34cf018b5e26daf6bde44f7f748da57a889c373435658cfb085ff537f3a683433c6d3

data/CHANGELOG.md

@@ -2,6 +2,36 @@
 
 All changes to the Agoo gem are documented here. Releases follow semantic versioning.
 
+## [2.15.2] - 2022-06-21
+
+### Fixed
+
+- Request no longer crashes when `#env` is not called before `#set`.
+
+## [2.15.1] - 2022-06-18
+
+### Added
+
+- Introspection can now be disabled with the `hide_schema` option to the Agoo server.
+
+- If an exception raised from a GraphQL callback responds to `code`
+  that code will be used as the HTTP status.
+
+- Added missing ability to set or add elements of a request argument
+  on GraphQL callback methods.
+
+## [2.15.0] - 2022-05-20
+
+### Added
+
+- Support added for PATCH.
+
+- A `:hide_schema` option has been added to show the graphql/schema as
+  not found unless added by with the handle method of the server.
+
+- Raising an exception that responds to `code` in a graphql resolve
+  function will return that code as the HTTP status code.
+
 ## [2.14.3] - 2022-05-05
 
 ### Fixed

data/ext/agoo/con.c

@@ -327,6 +327,8 @@ con_header_read(agooCon c, size_t *mlenp) {
 	    method = AGOO_PUT;
 	} else if (4 == b - c->buf && 0 == strncmp("POST", c->buf, 4)) {
 	    method = AGOO_POST;
+	} else if (5 == b - c->buf && 0 == strncmp("PATCH", c->buf, 5)) {
+	    method = AGOO_PATCH;
 	} else {
 	    return bad_request(c, 400, __LINE__);
 	}

data/ext/agoo/gqleval.c

@@ -478,8 +478,14 @@ gql_eval_get_hook(agooReq req) {
 	result = gql_doc_eval_func(&err, doc);
     }
     if (NULL == result) {
+	int code = 500;
+
+	if (err.code < 0) {
+	    code = -err.code;
+	}
+	err.code = AGOO_ERR_EVAL;
 	gql_doc_destroy(doc);
-	err_resp(req->res, &err, 500);
+	err_resp(req->res, &err, code);
 	return;
     }
     if (GQL_SUBSCRIPTION == doc->op->kind) {
@@ -648,7 +654,13 @@ gql_eval_post_hook(agooReq req) {
 	indent = (int)strtol(s, NULL, 10);
     }
     if (NULL == (result = eval_post(&err, req)) && AGOO_ERR_OK != err.code) {
-	err_resp(req->res, &err, 400);
+	int code = 400;
+
+	if (err.code < 0) {
+	    code = -err.code;
+	}
+	err.code = AGOO_ERR_EVAL;
+	err_resp(req->res, &err, code);
     } else if (NULL == result) {
 	value_resp(req, result, 200, indent);
     } else {

data/ext/agoo/request.c

@@ -684,10 +684,38 @@ call(VALUE self) {
     return io;
 }
 
+/* Document-method: set
+ *
+ * call-seq: set()
+ *
+ * Sets an key value pair to the environment of the request.
+ */
+static VALUE
+set(VALUE self, VALUE key, VALUE val) {
+    agooReq	r = DATA_PTR(self);
+
+    if (NULL == r) {
+	rb_raise(rb_eArgError, "Request is no longer valid.");
+    }
+    request_env(r, self);
+    rb_hash_aset((VALUE)r->env, key, val);
+
+    return Qnil;
+}
+
+static void
+request_mark(void *ptr) {
+    agooReq	r = (agooReq)ptr;
+
+    if (NULL != r->env && Qnil != (VALUE)r->env) {
+	rb_gc_mark((VALUE)r->env);
+    }
+}
+
 VALUE
 request_wrap(agooReq req) {
     // freed from the C side of things
-    return Data_Wrap_Struct(req_class, NULL, NULL, req);
+    return Data_Wrap_Struct(req_class, request_mark, NULL, req);
 }
 
 /* Document-class: Agoo::Request
@@ -723,6 +751,7 @@ request_init(VALUE mod) {
     rb_define_method(req_class, "body", body, 0);
     rb_define_method(req_class, "rack_logger", rack_logger, 0);
     rb_define_method(req_class, "call", call, 0);
+    rb_define_method(req_class, "set", set, 2);
 
     new_id = rb_intern("new");
 

data/ext/agoo/rgraphql.c

@@ -58,8 +58,24 @@ make_ruby_use(agooErr    err,
     volatile VALUE	v;
     ID			m = rb_intern(method);
 
-    if (!rb_respond_to(root, m) ||
-	Qnil == (v = rb_funcall(root, m, 0))) {
+    if (!rb_respond_to(root, m)) {
+	return AGOO_ERR_OK;
+    }
+    switch (rb_obj_method_arity(root, m)) {
+    case 0:
+	v = rb_funcall(root, m, 0);
+	break;
+    case 1:
+	v = rb_funcall(root, m, 1, Qnil);
+	break;
+    case 2:
+	v = rb_funcall(root, m, 2, Qnil, Qnil);
+	break;
+    default:
+	v = Qnil;
+	break;
+    }
+    if (Qnil == v) {
 	return AGOO_ERR_OK;
     }
     if (NULL == (type = gql_type_get(type_name))) {
@@ -89,7 +105,13 @@ rescue_error(VALUE x, VALUE ignore) {
     const char		*ms = rb_string_value_ptr(&msg);
 
     agoo_err_set(eval->err, AGOO_ERR_EVAL, "%s: %s", classname, ms);
+    if (rb_respond_to(info, rb_intern("code"))) {
+	VALUE	code = rb_funcall(info, rb_intern("code"), 0);
 
+	if (RUBY_T_FIXNUM == rb_type(code)) {
+	    eval->err->code = -FIX2INT(code);
+	}
+    }
     return Qfalse;
 }
 

data/ext/agoo/rserver.c

@@ -48,6 +48,7 @@ static VALUE	options_sym;
 static VALUE	post_sym;
 static VALUE	push_env_key;
 static VALUE	put_sym;
+static VALUE	patch_sym;
 
 static VALUE	rserver;
 
@@ -207,12 +208,12 @@ configure(agooErr err, int port, const char *root, VALUE options) {
 	}
 	if (Qnil != (v = rb_hash_lookup(options, ID2SYM(rb_intern("graphql"))))) {
 	    const char	*path;
-	    agooHook	dump_hook;
 	    agooHook	get_hook;
 	    agooHook	post_hook;
 	    agooHook	options_hook;
-	    char	schema_path[256];
+	    agooHook	head = NULL;
 	    long	plen;
+	    VALUE	hide_schema = Qnil;
 
 	    rb_check_type(v, T_STRING);
 	    if (AGOO_ERR_OK != gql_init(err)) {
@@ -220,21 +221,29 @@ configure(agooErr err, int port, const char *root, VALUE options) {
 	    }
 	    path = StringValuePtr(v);
 	    plen = (long)RSTRING_LEN(v);
-	    if ((int)sizeof(schema_path) - 8 < plen) {
-		rb_raise(rb_eArgError, "A graphql schema path is limited to %d characters.", (int)(sizeof(schema_path) - 8));
-	    }
-	    memcpy(schema_path, path, plen);
-	    memcpy(schema_path + plen, "/schema", 8);
 
-	    dump_hook = agoo_hook_func_create(AGOO_GET, schema_path, gql_dump_hook, &agoo_server.eval_queue);
 	    get_hook = agoo_hook_func_create(AGOO_GET, path, gql_eval_get_hook, &agoo_server.eval_queue);
 	    post_hook = agoo_hook_func_create(AGOO_POST, path, gql_eval_post_hook, &agoo_server.eval_queue);
 	    options_hook = agoo_hook_func_create(AGOO_OPTIONS, path, gql_eval_options_hook, &agoo_server.eval_queue);
-	    dump_hook->next = get_hook;
+	    if (Qnil != (hide_schema = rb_hash_lookup(options, ID2SYM(rb_intern("hide_schema")))) && Qtrue == hide_schema) {
+		head = get_hook;
+	    } else {
+		char		schema_path[256];
+		agooHook	dump_hook;
+
+		if ((int)sizeof(schema_path) - 8 < plen) {
+		    rb_raise(rb_eArgError, "A graphql schema path is limited to %d characters.", (int)(sizeof(schema_path) - 8));
+		}
+		memcpy(schema_path, path, plen);
+		memcpy(schema_path + plen, "/schema", 8);
+		dump_hook = agoo_hook_func_create(AGOO_GET, schema_path, gql_dump_hook, &agoo_server.eval_queue);
+		dump_hook->next = get_hook;
+		head = dump_hook;
+	    }
 	    get_hook->next = post_hook;
 	    post_hook->next = options_hook;
 	    options_hook->next = agoo_server.hooks;
-	    agoo_server.hooks = dump_hook;
+	    agoo_server.hooks = head;
 	}
 	if (Qnil != (v = rb_hash_lookup(options, ID2SYM(rb_intern("quiet"))))) {
 	    if (Qtrue == v) {
@@ -295,6 +304,8 @@ configure(agooErr err, int port, const char *root, VALUE options) {
  *   - *:ssl_sert* [_String_] filepath to the SSL certificate file.
  *
  *   - *:ssl_key* [_String_] filepath to the SSL private key file.
+ *
+ *   - *:hide_schema* [_true_|_false_] if true the graphql/schema path is handled.
  */
 static VALUE
 rserver_init(int argc, VALUE *argv, VALUE self) {
@@ -971,6 +982,8 @@ handle(VALUE self, VALUE method, VALUE pattern, VALUE handler) {
 	meth = AGOO_POST;
     } else if (put_sym == method) {
 	meth = AGOO_PUT;
+    } else if (patch_sym == method) {
+	meth = AGOO_PATCH;
     } else if (Qnil == method) {
 	meth = AGOO_ALL;
     } else {
@@ -1277,6 +1290,7 @@ server_init(VALUE mod) {
     options_sym = ID2SYM(rb_intern("OPTIONS"));		rb_gc_register_address(&options_sym);
     post_sym = ID2SYM(rb_intern("POST"));		rb_gc_register_address(&post_sym);
     put_sym = ID2SYM(rb_intern("PUT"));			rb_gc_register_address(&put_sym);
+    patch_sym = ID2SYM(rb_intern("PATCH"));		rb_gc_register_address(&patch_sym);
 
     push_env_key = rb_str_new_cstr("rack.upgrade");	rb_gc_register_address(&push_env_key);
 

data/lib/agoo/version.rb

@@ -1,5 +1,5 @@
 
 module Agoo
   # Agoo version.
-  VERSION = '2.14.3'
+  VERSION = '2.15.2'
 end

data/test/rack_handler_test.rb

@@ -33,7 +33,7 @@ size=big
 	]
       elsif 'POST' == req['REQUEST_METHOD']
 	[ 204, { }, [] ]
-      elsif 'PUT' == req['REQUEST_METHOD']
+      elsif 'PUT' == req['REQUEST_METHOD'] ||  'PATCH' == req['REQUEST_METHOD']
 	[ 201,
 	  { },
 	  [ req['rack.input'].read ]
@@ -65,6 +65,7 @@ size=big
     Agoo::Server.handle(:GET, "/tellme", handler)
     Agoo::Server.handle(:POST, "/makeme", handler)
     Agoo::Server.handle(:PUT, "/makeme", handler)
+    Agoo::Server.handle(:PATCH, "/makeme", handler)
 
     Agoo::Server.start()
 
@@ -156,4 +157,20 @@ size=big
     assert_equal('hello', res.body)
   end
 
+  def test_patch
+    uri = URI('http://localhost:6467/makeme')
+    req = Net::HTTP::Patch.new(uri)
+    # Set the headers the way we want them.
+    req['Accept-Encoding'] = '*'
+    req['Accept'] = 'application/json'
+    req['User-Agent'] = 'Ruby'
+    req.body = 'hello'
+
+    res = Net::HTTP.start(uri.hostname, uri.port) { |h|
+      h.request(req)
+    }
+    assert_equal(Net::HTTPCreated, res.class)
+    assert_equal('hello', res.body)
+  end
+
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: agoo
 version: !ruby/object:Gem::Version
-  version: 2.14.3
+  version: 2.15.2
 platform: ruby
 authors:
 - Peter Ohler
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-05-05 00:00:00.000000000 Z
+date: 2022-06-21 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: oj