agoo 2.15.2 → 2.15.4

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: ad246b7aa0e634eff81c6c9bdec4d3567115a3beb8b6c237e25f87215fc2dbe3
-  data.tar.gz: 03b6217fd351024938607dd036dcfd437a0dd967b22eca2b51aa0ed8a041d969
+  metadata.gz: f4f9e647ee51b00410d85de35a301d61dd6795175a2add858d29b4777d5889fb
+  data.tar.gz: 707f1a41efc1f051c15f661774063b45e48f989377d14c9c5cf59579fb962f68
 SHA512:
-  metadata.gz: a244ea77e0f13688e2e6237c31eb320af4c0eeb9780f6023ef6337db7f64839ce543a27d5890155fb0908f0ce6567733b750be0d7b7c4de246430c54a736b6f4
-  data.tar.gz: abf7f274c15852b21b64ac6018b03264d08968902b9ad71d4f56d84a5ea34cf018b5e26daf6bde44f7f748da57a889c373435658cfb085ff537f3a683433c6d3
+  metadata.gz: 896d35cc2800c9096192cf86d8595c1bf01f60f4b6509283d059911dbbc29ba5062fd03c38356d3bd29860d71a2f050d7c6ac5d12f011de699f8e5a9c33615af
+  data.tar.gz: 140970f8d879fd73773688f3d1cb0b526834abe20077aa6b5e5b32e3c82c83359f74b425fc130df0ed64ac8727f41e74a69222d26eab9940d94640f91874dc92

data/CHANGELOG.md

@@ -2,6 +2,19 @@
 
 All changes to the Agoo gem are documented here. Releases follow semantic versioning.
 
+## [2.15.4] - 2023-03-05
+
+### Fixed
+
+- Ruby 3.2.1 and MacOS didn't work well together with libssl missing
+  some symbols. Included libcrypt to fix that.
+
+## [2.15.3] - 2022-09-23
+
+### Fixed
+
+- Asset names with spaces or special characters are now loaded correctly.
+
 ## [2.15.2] - 2022-06-21
 
 ### 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/github/workflow/status/ohler55/agoo/CI?logo=github)](https://github.com/ohler55/agoo/actions/workflows/CI.yml)
+[![Build Status](https://img.shields.io/github/actions/workflow/status/ohler55/agoo/CI.yml?logo=github&branch=develop)](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)
 

data/ext/agoo/extconf.rb

@@ -23,6 +23,7 @@ have_header('stdatomic.h')
 have_header('sys/epoll.h')
 have_header('openssl/ssl.h')
 have_library('ssl')
+have_library('crypto')
 
 create_makefile(File.join(extension_name, extension_name))
 

data/ext/agoo/page.c

@@ -110,6 +110,16 @@ static struct _mime	mime_map[] = {
 
 static const char	page_fmt[] = "HTTP/1.1 200 OK\r\nContent-Type: %s\r\nContent-Length: %ld\r\n\r\n";
 static const char	page_min_fmt[] = "HTTP/1.1 200 OK\r\nContent-Length: %ld\r\n";
+// 0123456789abcdef0123456789abcdef
+static const char hex_map[] = "\
+................................\
+................\x00\x01\x02\x03\x04\x05\x06\x07\x08\x09......\
+.\x0a\x0b\x0c\x0d\x0e\x0f.........................\
+.\x0a\x0b\x0c\x0d\x0e\x0f.........................\
+................................\
+................................\
+................................\
+................................";
 
 static struct _cache	cache = {
     .buckets = {0},
@@ -120,29 +130,35 @@ static struct _cache	cache = {
     .head_rules = NULL,
 };
 
+static char
+parse_percent_seq(const char *seq) {
+    const char  *end = seq + 2;
+    uint8_t        h;
+    uint8_t        b = 0;
+
+    for (; seq < end; seq++) {
+        if (16 < (h = hex_map[(uint8_t)*seq])) {
+            return '?';
+        }
+        b = (b << 4) | h;
+    }
+    return (char)b;
+}
+
 static uint64_t
 calc_hash(const char *key, int *lenp) {
     int			len = 0;
     int			klen = *lenp;
     uint64_t		h = 0;
-    bool		special = false;
     const uint8_t	*k = (const uint8_t*)key;
 
     for (; len < klen; k++) {
-	// narrow to most used range of 0x4D (77) in size
-	if (*k < 0x2D || 0x7A < *k) {
-	    special = true;
-	}
 	// fast, just spread it out
 	h = 77 * h + (*k - 0x2D);
 	len++;
     }
+    *lenp = len;
 
-    if (special) {
-	*lenp = -len;
-    } else {
-	*lenp = len;
-    }
     return h;
 }
 
@@ -790,9 +806,25 @@ agoo_page_get(agooErr err, const char *path, int plen, const char *root) {
 	if ('/' != *root && '/' != *(s - 1) && '/' != *path) {
 	    *s++ = '/';
 	}
-	strncpy(s, path, plen);
-	s[plen] = '\0';
-
+        // TBD if path has % then ...
+        if (NULL != memchr(path, '%', plen)) {
+            const char  *pend = path + plen;
+            const char  *pp = path;
+
+            for (; pp < pend; pp++) {
+                if ('%' != *pp) {
+                    *s++ = *pp;
+                    continue;
+                }
+                *s++ = parse_percent_seq(pp+1);
+                pp += 2;
+            }
+        } else {
+            strncpy(s, path, plen);
+            s += plen;
+        }
+	*s = '\0';
+        plen = (int)(s - full_path);
 	if (NULL == (page = cache_root_get(full_path, plen))) {
 	    if (NULL != cache.root) {
 		agooPage	old;
@@ -815,10 +847,12 @@ agoo_page_get(agooErr err, const char *path, int plen, const char *root) {
 	}
     } else {
 	if (NULL == (page = cache_get(path, plen))) {
-	    if (NULL != cache.root) {
+            bool        has_percent = NULL != memchr(path, '%', plen);
+
+	    if (NULL != cache.root || has_percent) {
 		agooPage	old;
-		char	full_path[2048];
-		char	*s = stpcpy(full_path, cache.root);
+		char	        full_path[2048];
+		char	        *s = stpcpy(full_path, cache.root);
 
 		if ('/' != *cache.root && '/' != *path) {
 		    *s++ = '/';
@@ -827,17 +861,34 @@ agoo_page_get(agooErr err, const char *path, int plen, const char *root) {
 		    AGOO_ERR_MEM(err, "Page path");
 		    return NULL;
 		}
-		strncpy(s, path, plen);
-		s[plen] = '\0';
-		if (NULL == (page = agoo_page_create(full_path))) {
+                if (has_percent) {
+                    const char  *pend = path + plen;
+                    const char  *pp = path;
+
+                    for (; pp < pend; pp++) {
+                        if ('%' != *pp) {
+                            *s++ = *pp;
+                            continue;
+                        }
+                        *s++ = parse_percent_seq(pp+1);
+                        pp += 2;
+                    }
+                    *s = '\0';
+                } else {
+                    strncpy(s, path, plen);
+                    s[plen] = '\0';
+                }
+		if (NULL == (page = agoo_page_create(full_path))) { // TBD full_path or original path?
 		    AGOO_ERR_MEM(err, "Page");
 		    return NULL;
 		}
+                plen = (int)strlen(full_path);
 		if (!update_contents(page) || NULL == page->resp) {
 		    agoo_page_destroy(page);
 		    agoo_err_set(err, AGOO_ERR_NOT_FOUND, "not found.");
 		    return NULL;
 		}
+                // Cache key is the original path/plen.
 		if (NULL != (old = cache_set(path, plen, page))) {
 		    agoo_page_destroy(old);
 		}

data/lib/agoo/version.rb

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

data/test/static_test.rb

@@ -69,6 +69,25 @@ class StaticTest < Minitest::Test
     assert_equal(expect, content)
   end
 
+  def test_fetch_space_name
+    uri = URI('http://localhost:6469/space%20in%20name.html')
+    expect = %|<!DOCTYPE html>
+<html>
+  <head><title>Agoo Space Test</title></head>
+  <body>Agoo</body>
+</html>
+|
+    req = Net::HTTP::Get.new(uri)
+    req['Accept-Encoding'] = '*'
+    req['User-Agent'] = 'Ruby'
+    res = Net::HTTP.start(uri.hostname, uri.port) { |h|
+      h.request(req)
+    }
+    content = res.body
+    assert_equal('text/html', res['Content-Type'])
+    assert_equal(expect, content)
+  end
+
   def test_mime
     uri = URI('http://localhost:6469/odd.odd')
     req = Net::HTTP::Get.new(uri)

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: agoo
 version: !ruby/object:Gem::Version
-  version: 2.15.2
+  version: 2.15.4
 platform: ruby
 authors:
 - Peter Ohler
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-06-21 00:00:00.000000000 Z
+date: 2023-03-05 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: oj
@@ -190,7 +190,7 @@ required_rubygems_version: !ruby/object:Gem::Requirement
       version: '0'
 requirements:
 - Linux or macOS
-rubygems_version: 3.3.3
+rubygems_version: 3.4.1
 signing_key:
 specification_version: 4
 summary: An HTTP server