agoo 2.14.1 → 2.15.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0fbaa8a1e2f42cda657178256643bcf0161f93d9be09e90512084eb877b12aa9
-  data.tar.gz: e33135684316f9e0244f0cc2e781561f99374888c7756866e0d2ed33db1c0446
+  metadata.gz: 1774bcd29b45056263ad986722015b99efa5c2bc8b0607b3a08afb79cd40cd70
+  data.tar.gz: dddc4062987102c87f1e2556ad425b33f3e0dd0a380c2a2416348ba2e07869d2
 SHA512:
-  metadata.gz: 630cf6226a74119dbb5417a8a33df2893e08e61acd6037f61d6d6a1d424c22a7bb61bfa59c65420b3e4026a421764b128b36ca9ae5dc2087722f338228f2e732
-  data.tar.gz: 6798931eaa91d29ccc0e6abe701cc62cc44b129b22e34f0f0dc53e26f409ca27c7a001424197f9c4c2f02820a8a670e38609e408ca4b6e5069957c9981a6b49f
+  metadata.gz: 5b4dbbe0b330c4456d46354019940e962a79229486b323940e20d3b8213416fd025d6279772b35f441bbbdd8e1fa5cfebe812ecba7f5d7e3a5ad16d46b41ef25
+  data.tar.gz: aad80f64bc75f27663c1372d02ced4a205ea5e89a5deba77619951cc09d0a083810f57520375a9fc8fe658e591f453367e1b4638f362ccfc95a2bbba3a183c7a

data/CHANGELOG.md

@@ -2,6 +2,24 @@
 
 All changes to the Agoo gem are documented here. Releases follow semantic versioning.
 
+## [2.15.0] - 2022-05-20
+
+### Added
+
+- Support added for PATCH.
+
+## [2.14.3] - 2022-05-05
+
+### Fixed
+- Agoo now reports an error if the developer make the mistake of
+  building a schema that loops back on itself too many times using
+  fragments.
+
+## [2.14.2] - 2022-02-22
+
+### Fixed
+- Invalid SDL now raises and exception instead of crashing.
+
 ## [2.14.1] - 2021-06-09
 
 ### Fixed

data/README.md

@@ -1,6 +1,6 @@
 # [![{}j](misc/agoo_128.svg)](http://www.ohler.com/agoo) Agoo
 
-[![Build Status](https://img.shields.io/travis/ohler55/agoo/master.svg)](http://travis-ci.org/ohler55/agoo?branch=master)
+[![Build Status](https://img.shields.io/github/workflow/status/ohler55/agoo/CI?logo=github)](https://github.com/ohler55/agoo/actions/workflows/CI.yml)
 [![Gem Version](https://badge.fury.io/rb/agoo.svg)](https://badge.fury.io/rb/agoo)
 ![Gem](https://img.shields.io/gem/dt/agoo.svg) [![TideLift](https://tidelift.com/badges/github/ohler55/agoo)](https://tidelift.com/subscription/pkg/rubygems-agoo?utm_source=rubygems-agoo&utm_medium=referral&utm_campaign=readme)
 
@@ -163,4 +163,3 @@ the develop branch. Pull requests should be made against the develop branch.
  - *WABuR* *repo*: https://github.com/ohler55/wabur has an option to use Agoo
 
  - *Perfer* *repo*: https://github.com/ohler55/perfer
-

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/extconf.rb

@@ -20,7 +20,7 @@ CONFIG['warnflags'].slice!(/ -Wdeclaration-after-statement/)
 CONFIG['warnflags'].slice!(/ -Wmissing-noreturn/)
 
 have_header('stdatomic.h')
-#have_header('sys/epoll.h')
+have_header('sys/epoll.h')
 have_header('openssl/ssl.h')
 have_library('ssl')
 

data/ext/agoo/gqleval.c

@@ -21,6 +21,7 @@
 #include "websocket.h"
 
 #define MAX_RESOLVE_ARGS	16
+#define MAX_DEPTH		100
 
 gqlRef		gql_root = NULL;
 gqlType		_gql_root_type = NULL;
@@ -273,7 +274,10 @@ gql_eval_sels(agooErr err, gqlDoc doc, gqlRef ref, gqlField field, gqlSel sels,
     gqlSel	sel;
     gqlField	sf = NULL;
 
-    // TBD if depth over max then return an error
+    if (MAX_DEPTH < depth) {
+	return agoo_err_set(err, AGOO_ERR_EVAL, "Maximum resolve depth of %d exceeded.", MAX_DEPTH);
+    }
+    depth++;
 
     for (sel = sels; NULL != sel; sel = sel->next) {
 	if (NULL != field) {

data/ext/agoo/gqlintro.c

@@ -1497,13 +1497,13 @@ gql_intro_eval(agooErr err, gqlDoc doc, gqlSel sel, gqlValue result, int depth)
     struct _gqlCobj	obj;
 
     if (0 == strcmp("__type", sel->name)) {
-	if (1 < depth) {
+	if (2 < depth) {
 	    return agoo_err_set(err, AGOO_ERR_EVAL, "__type can only be called from a query root.");
 	}
 	obj.clas = &root_class;
 	obj.ptr = NULL;
     } else if (0 == strcmp("__schema", sel->name)) {
-	if (1 < depth) {
+	if (2 < depth) {
 	    return agoo_err_set(err, AGOO_ERR_EVAL, "__scheme can only be called from a query root.");
 	}
 	obj.clas = &root_class;

data/ext/agoo/graphql.c

@@ -892,8 +892,8 @@ gql_assure_nonnull(agooErr err, gqlType base) {
 
 void
 gql_type_destroy(gqlType type) {
-    type_destroy(type);
     type_remove(type);
+    type_destroy(type);
 }
 
 // If negative then there are non-simple-string characters.

data/ext/agoo/ready.c

@@ -5,7 +5,7 @@
 #include <string.h>
 #include <unistd.h>
 
-#if HAVE_SYS_EPOLL_H
+#ifdef HAVE_SYS_EPOLL_H
 #include <sys/epoll.h>
 #else
 #include <ctype.h>
@@ -22,7 +22,7 @@
 // milliseconds
 #define MAX_WAIT		10
 
-#if HAVE_SYS_EPOLL_H
+#ifdef HAVE_SYS_EPOLL_H
 #define EPOLL_SIZE		100
 #else
 #define INITIAL_POLL_SIZE	1024
@@ -34,7 +34,7 @@ typedef struct _link {
     int			fd;
     void		*ctx;
     agooHandler		handler;
-#if HAVE_SYS_EPOLL_H
+#ifdef HAVE_SYS_EPOLL_H
     uint32_t		events; // last events set
 #else
     struct pollfd	*pp;
@@ -45,7 +45,7 @@ struct _agooReady {
     Link	links;
     int		lcnt;
     double	next_check;
-#if HAVE_SYS_EPOLL_H
+#ifdef HAVE_SYS_EPOLL_H
     int		epoll_fd;
 #else
     struct pollfd	*pa;
@@ -82,7 +82,7 @@ agoo_ready_create(agooErr err) {
 	ready->links = NULL;
 	ready->lcnt = 0;
 	ready->next_check = dtime() + CHECK_FREQ;
-#if HAVE_SYS_EPOLL_H
+#ifdef HAVE_SYS_EPOLL_H
 	if (0 > (ready->epoll_fd = epoll_create(1))) {
 	    agoo_err_no(err, "epoll create failed");
 	    return NULL;
@@ -111,7 +111,7 @@ agoo_ready_destroy(agooReady ready) {
 	}
 	AGOO_FREE(link);
     }
-#if HAVE_SYS_EPOLL_H
+#ifdef HAVE_SYS_EPOLL_H
     close(ready->epoll_fd);
 #else
     AGOO_FREE(ready->pa);
@@ -137,7 +137,7 @@ agoo_ready_add(agooErr		err,
     ready->links = link;
     ready->lcnt++;
 
-#if HAVE_SYS_EPOLL_H
+#ifdef HAVE_SYS_EPOLL_H
     link->events = EPOLLIN;
     {
 	struct epoll_event	event = {
@@ -181,7 +181,7 @@ ready_remove(agooReady ready, Link link) {
     if (NULL != link->next) {
 	link->next->prev = link->prev;
     }
-#if HAVE_SYS_EPOLL_H
+#ifdef HAVE_SYS_EPOLL_H
     {
 	struct epoll_event	event = {
 	    .events = 0,
@@ -214,7 +214,7 @@ agoo_ready_go(agooErr err, agooReady ready) {
     Link	link;
     Link	next;
 
-#if HAVE_SYS_EPOLL_H
+#ifdef HAVE_SYS_EPOLL_H
     struct epoll_event	events[EPOLL_SIZE];
     struct epoll_event	*ep;
     int			cnt;

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;
 
@@ -523,14 +524,14 @@ handle_rack_inner(VALUE x) {
 		if (Qnil != body) {
 		    body = rb_ivar_get(body, rb_intern("@body"));
 		}
-		if (rb_respond_to(body, rb_intern("each"))) {
-		    rb_iterate(rb_each, body, body_len_cb, (VALUE)&bsize);
+		if (rb_respond_to(body, each_id)) {
+		    rb_block_call(body, each_id, 0, 0, body_len_cb, (VALUE)&bsize);
 		}
 	    } else {
-		rb_iterate(rb_each, bv, body_len_cb, (VALUE)&bsize);
+		rb_block_call(bv, each_id, 0, 0, body_len_cb, (VALUE)&bsize);
 	    }
 	} else {
-	    rb_iterate(rb_each, bv, body_len_cb, (VALUE)&bsize);
+	    rb_block_call(bv, each_id, 0, 0, body_len_cb, (VALUE)&bsize);
 	}
     }
     switch (code) {
@@ -587,7 +588,7 @@ handle_rack_inner(VALUE x) {
 	if (T_HASH == rb_type(hv)) {
 	    rb_hash_foreach(hv, header_cb, (VALUE)&t);
 	} else {
-	    rb_iterate(rb_each, hv, header_each_cb, (VALUE)&t);
+	    rb_block_call(hv, each_id, 0, 0, header_each_cb, (VALUE)&t);
 	}
     }
     t = agoo_text_append(t, "\r\n", 2);
@@ -602,7 +603,7 @@ handle_rack_inner(VALUE x) {
 		t = agoo_text_append(t, StringValuePtr(v), (int)RSTRING_LEN(v));
 	    }
 	} else {
-	    rb_iterate(rb_each, bv, body_append_cb, (VALUE)&t);
+	    rb_block_call(bv, each_id, 0, 0, body_append_cb, (VALUE)&t);
 	}
     }
     agoo_res_message_push(req->res, t);
@@ -971,6 +972,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 +1280,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.1'
+  VERSION = '2.15.0'
 end

data/test/graphql_error_test.rb

@@ -0,0 +1,35 @@
+#!/usr/bin/env ruby
+
+$: << File.dirname(__FILE__)
+$root_dir = File.dirname(File.expand_path(File.dirname(__FILE__)))
+%w(lib ext).each do |dir|
+  $: << File.join($root_dir, dir)
+end
+
+require 'minitest'
+require 'minitest/autorun'
+
+require 'agoo'
+
+class Schema
+  attr_reader :query
+
+  def initialize()
+  end
+end
+
+class GraphQLErrTest < Minitest::Test
+
+  def test_bad_sdl
+    bad_sdl = %^
+type Query @ruby(class: "Query" {
+}
+^
+    Agoo::GraphQL.schema(Schema.new) {
+      assert_raises(StandardError) {
+	Agoo::GraphQL.load(bad_sdl)
+      }
+    }
+  end
+
+end

data/test/graphql_test.rb

@@ -458,6 +458,32 @@ fragment basic on Artist {
     post_test(uri, body, 'application/graphql', expect)
   end
 
+  def test_post_fragment_loop
+    uri = URI('http://localhost:6472/graphql?indent=2')
+    body = %^
+{
+  artist(name:"Fazerdaze") {
+    ...loop
+  }
+}
+
+fragment loop on Artist {
+  name
+  ...loop
+}
+^
+    expect = %^{
+  "errors":[
+    {
+      "message":"Maximum resolve depth of 100 exceeded.",
+      "code":"eval error"
+    }
+  ]
+}
+^
+    post_test(uri, body, 'application/graphql', expect, 'errors.0.timestamp')
+  end
+
   def test_post_json_fragment
     uri = URI('http://localhost:6472/graphql?indent=2')
     body = %^{
@@ -1044,7 +1070,7 @@ mutation {
     assert_equal(expect, content)
   end
 
-  def post_test(uri, body, content_type, expect)
+  def post_test(uri, body, content_type, expect, ignore=nil)
     uri = URI(uri)
     req = Net::HTTP::Post.new(uri)
     req['Accept-Encoding'] = '*'
@@ -1055,6 +1081,11 @@ mutation {
     }
     content = res.body
     assert_equal('application/json', res['Content-Type'])
+    unless ignore.nil?
+      result = Oj.load(content, mode: :strict)
+      deep_delete(result, ignore.split('.'))
+      content = Oj.dump(result, indent: 2)
+    end
     assert_equal(expect, content)
   end
 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.1
+  version: 2.15.0
 platform: ruby
 authors:
 - Peter Ohler
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-06-09 00:00:00.000000000 Z
+date: 2022-05-20 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: oj
@@ -154,6 +154,7 @@ files:
 - test/bind_test.rb
 - test/domain_test.rb
 - test/early_hints_test.rb
+- test/graphql_error_test.rb
 - test/graphql_test.rb
 - test/hijack_test.rb
 - test/log_test.rb
@@ -189,7 +190,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements:
 - Linux or macOS
-rubygems_version: 3.2.3
+rubygems_version: 3.3.3
 signing_key:
 specification_version: 4
 summary: An HTTP server
@@ -198,6 +199,7 @@ test_files:
 - test/bind_test.rb
 - test/domain_test.rb
 - test/early_hints_test.rb
+- test/graphql_error_test.rb
 - test/graphql_test.rb
 - test/hijack_test.rb
 - test/log_test.rb