agoo 2.14.1 → 2.14.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 0fbaa8a1e2f42cda657178256643bcf0161f93d9be09e90512084eb877b12aa9
-  data.tar.gz: e33135684316f9e0244f0cc2e781561f99374888c7756866e0d2ed33db1c0446
+  metadata.gz: e538b3f1c7ed9ea23d8529c29bf5104d0882b0dcda49e6114bb011491b1e3ed7
+  data.tar.gz: ee1a1284779ec7108675794f4eff89f826c76f3a5e10a7f07a42bab8bbae31cb
 SHA512:
-  metadata.gz: 630cf6226a74119dbb5417a8a33df2893e08e61acd6037f61d6d6a1d424c22a7bb61bfa59c65420b3e4026a421764b128b36ca9ae5dc2087722f338228f2e732
-  data.tar.gz: 6798931eaa91d29ccc0e6abe701cc62cc44b129b22e34f0f0dc53e26f409ca27c7a001424197f9c4c2f02820a8a670e38609e408ca4b6e5069957c9981a6b49f
+  metadata.gz: e28b50c1bea0934691679719532108fdd6d912f3beed8b9eabe6071ccd71af393b14f3c81d094cb82b5777a22884cc716e12d3c66629e39c5f7f1290919fc030
+  data.tar.gz: 1d478668d82b79acc3d2cabb56fe9aca55fb3513b4c5a5810c0d200a69b081573ff532a346c13d513fae0980fa6331f3701ee041d43ea7cfc32cd2d2a586d0c2

data/CHANGELOG.md

@@ -2,6 +2,11 @@
 
 All changes to the Agoo gem are documented here. Releases follow semantic versioning.
 
+## [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/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/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

@@ -523,14 +523,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 +587,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 +602,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);

data/lib/agoo/version.rb

@@ -1,5 +1,5 @@
 
 module Agoo
   # Agoo version.
-  VERSION = '2.14.1'
+  VERSION = '2.14.2'
 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

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: agoo
 version: !ruby/object:Gem::Version
-  version: 2.14.1
+  version: 2.14.2
 platform: ruby
 authors:
 - Peter Ohler
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2021-06-09 00:00:00.000000000 Z
+date: 2022-02-22 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