iodine 0.7.14 → 0.7.15

data/ext/iodine/fio.h

@@ -109,7 +109,7 @@ Version and helper macros
 #define FIO_VERSION_MAJOR 0
 #define FIO_VERSION_MINOR 7
 #define FIO_VERSION_PATCH 0
-#define FIO_VERSION_BETA 3
+#define FIO_VERSION_BETA 4
 
 /* Automatically convert version data to a string constant - ignore these two */
 #define FIO_MACRO2STR_STEP2(macro) #macro

data/ext/iodine/fio_cli.c

@@ -91,10 +91,11 @@ char const *fio_cli_get_line_type(fio_cli_parser_data_s *parser,
   char const **pos = parser->names;
   while (*pos) {
     switch ((intptr_t)*pos) {
-    case /* FIO_CLI_TYPE_STRING */ 0x1: /* fallthrough */
-    case /* FIO_CLI_TYPE_BOOL */ 0x2:   /* fallthrough */
-    case /* FIO_CLI_TYPE_INT */ 0x3:    /* fallthrough */
-    case /* FIO_CLI_TYPE_PRINT */ 0x4:  /* fallthrough */
+    case /* FIO_CLI_TYPE_STRING */ 0x1:       /* fallthrough */
+    case /* FIO_CLI_TYPE_BOOL */ 0x2:         /* fallthrough */
+    case /* FIO_CLI_TYPE_INT */ 0x3:          /* fallthrough */
+    case /* FIO_CLI_TYPE_PRINT */ 0x4:        /* fallthrough */
+    case /* FIO_CLI_TYPE_PRINT_HEADER */ 0x5: /* fallthrough */
       ++pos;
       continue;
     }
@@ -106,10 +107,11 @@ char const *fio_cli_get_line_type(fio_cli_parser_data_s *parser,
   return NULL;
 found:
   switch ((size_t)pos[1]) {
-  case /* FIO_CLI_TYPE_STRING */ 0x1: /* fallthrough */
-  case /* FIO_CLI_TYPE_BOOL */ 0x2:   /* fallthrough */
-  case /* FIO_CLI_TYPE_INT */ 0x3:    /* fallthrough */
-  case /* FIO_CLI_TYPE_PRINT */ 0x4:  /* fallthrough */
+  case /* FIO_CLI_TYPE_STRING */ 0x1:       /* fallthrough */
+  case /* FIO_CLI_TYPE_BOOL */ 0x2:         /* fallthrough */
+  case /* FIO_CLI_TYPE_INT */ 0x3:          /* fallthrough */
+  case /* FIO_CLI_TYPE_PRINT */ 0x4:        /* fallthrough */
+  case /* FIO_CLI_TYPE_PRINT_HEADER */ 0x5: /* fallthrough */
     return pos[1];
   }
   return NULL;
@@ -196,7 +198,7 @@ error: /* handle errors*/
           (int)arg.len, arg.data, arg.len ? "with value" : "",
           value ? (value[0] ? value : "(empty)") : "(null)");
 print_help:
-  fprintf(stderr, "\n%s\n\n",
+  fprintf(stderr, "\n%s\n",
           parser->description ? parser->description
                               : "This application accepts any of the following "
                                 "possible arguments:");
@@ -204,10 +206,11 @@ print_help:
   char const **pos = parser->names;
   while (*pos) {
     switch ((intptr_t)*pos) {
-    case /* FIO_CLI_TYPE_STRING */ 0x1: /* fallthrough */
-    case /* FIO_CLI_TYPE_BOOL */ 0x2:   /* fallthrough */
-    case /* FIO_CLI_TYPE_INT */ 0x3:    /* fallthrough */
-    case /* FIO_CLI_TYPE_PRINT */ 0x4:  /* fallthrough */
+    case /* FIO_CLI_TYPE_STRING */ 0x1:       /* fallthrough */
+    case /* FIO_CLI_TYPE_BOOL */ 0x2:         /* fallthrough */
+    case /* FIO_CLI_TYPE_INT */ 0x3:          /* fallthrough */
+    case /* FIO_CLI_TYPE_PRINT */ 0x4:        /* fallthrough */
+    case /* FIO_CLI_TYPE_PRINT_HEADER */ 0x5: /* fallthrough */
       ++pos;
       continue;
     }
@@ -217,6 +220,10 @@ print_help:
       fprintf(stderr, "%s\n", pos[0]);
       pos += 2;
       continue;
+    case /* FIO_CLI_TYPE_PRINT_HEADER */ 0x5:
+      fprintf(stderr, "\n\x1B[4m%s\x1B[0m\n", pos[0]);
+      pos += 2;
+      continue;
 
     case /* FIO_CLI_TYPE_STRING */ 0x1: /* fallthrough */
     case /* FIO_CLI_TYPE_BOOL */ 0x2:   /* fallthrough */
@@ -321,14 +328,15 @@ void fio_cli_start AVOID_MACRO(int argc, char const *argv[], int unnamed_min,
   char const **line = names;
   while (*line) {
     switch ((intptr_t)*line) {
-    case /* FIO_CLI_TYPE_STRING */ 0x1: /* fallthrough */
-    case /* FIO_CLI_TYPE_BOOL */ 0x2:   /* fallthrough */
-    case /* FIO_CLI_TYPE_INT */ 0x3:    /* fallthrough */
-    case /* FIO_CLI_TYPE_PRINT */ 0x4:  /* fallthrough */
+    case /* FIO_CLI_TYPE_STRING */ 0x1:       /* fallthrough */
+    case /* FIO_CLI_TYPE_BOOL */ 0x2:         /* fallthrough */
+    case /* FIO_CLI_TYPE_INT */ 0x3:          /* fallthrough */
+    case /* FIO_CLI_TYPE_PRINT */ 0x4:        /* fallthrough */
+    case /* FIO_CLI_TYPE_PRINT_HEADER */ 0x5: /* fallthrough */
       ++line;
       continue;
     }
-    if (line[1] != FIO_CLI_TYPE_PRINT)
+    if (line[1] != FIO_CLI_TYPE_PRINT && line[1] != FIO_CLI_TYPE_PRINT_HEADER)
       fio_cli_map_line2alias(*line);
     ++line;
   }

data/ext/iodine/fio_cli.h

@@ -22,8 +22,10 @@ CLI API
 #define FIO_CLI_TYPE_BOOL ((char *)0x2)
 /** Indicates the previous CLI argument should be an Integer (numerical). */
 #define FIO_CLI_TYPE_INT ((char *)0x3)
-/** Indicates the previous CLI argument should be an Integer (numerical). */
+/** Indicates the previous CLI string should be printed as is. */
 #define FIO_CLI_TYPE_PRINT ((char *)0x4)
+/** Indicates the previous CLI string should be printed as a header. */
+#define FIO_CLI_TYPE_PRINT_HEADER ((char *)0x5)
 
 /**
  * This function parses the Command Line Interface (CLI), creating a temporary

data/ext/iodine/fiobj_data.c

@@ -867,11 +867,16 @@ static fio_str_info_s fiobj_data_pread_file(FIOBJ io, intptr_t start_at,
     start_at = 0;
   if (length + start_at > (uint64_t)size)
     length = size - start_at;
-  if (length == 0)
+  if (length == 0) {
+    /* free memory once there's no more data to read */
+    obj2io(io)->capa = 0;
+    fio_free(obj2io(io)->buffer);
+    obj2io(io)->buffer = NULL;
     return (fio_str_info_s){
         .data = NULL,
         .len = 0,
     };
+  }
   obj2io(io)->len = 0;
   obj2io(io)->pos = 0;
   fiobj_data_pre_write(io, length + 1);

data/ext/iodine/fiobj_mustache.c

@@ -73,20 +73,68 @@ FIOBJ fiobj_mustache_build(mustache_s *mustache, FIOBJ data) {
 Mustache Callbacks
 ***************************************************************************** */
 
-static inline FIOBJ fiobj_mustache_find_obj(mustache_section_s *section,
-                                            const char *name,
-                                            uint32_t name_len) {
+static inline FIOBJ fiobj_mustache_find_obj_absolute(FIOBJ parent, FIOBJ key) {
+  if (!FIOBJ_TYPE_IS(parent, FIOBJ_T_HASH))
+    return FIOBJ_INVALID;
+  FIOBJ o = FIOBJ_INVALID;
+  o = fiobj_hash_get(parent, key);
+  return o;
+}
+
+static inline FIOBJ fiobj_mustache_find_obj_tree(mustache_section_s *section,
+                                                 const char *name,
+                                                 uint32_t name_len) {
   FIOBJ key = fiobj_str_tmp();
   fiobj_str_write(key, name, name_len);
-  FIOBJ o = FIOBJ_INVALID;
   do {
-    if (!FIOBJ_TYPE_IS((FIOBJ)section->udata2, FIOBJ_T_HASH))
-      continue;
-    o = fiobj_hash_get((FIOBJ)section->udata2, key);
-    section = mustache_section_parent(section);
-  } while (o == FIOBJ_INVALID && section);
-  return o;
+    FIOBJ tmp = fiobj_mustache_find_obj_absolute((FIOBJ)section->udata2, key);
+    if (tmp != FIOBJ_INVALID) {
+      return tmp;
+    }
+  } while ((section = mustache_section_parent(section)));
+  return FIOBJ_INVALID;
 }
+
+static inline FIOBJ fiobj_mustache_find_obj(mustache_section_s *section,
+                                            const char *name,
+                                            uint32_t name_len) {
+  FIOBJ tmp = fiobj_mustache_find_obj_tree(section, name, name_len);
+  if (tmp != FIOBJ_INVALID)
+    return tmp;
+  /* interpolate sections... */
+  uint32_t dot = 0;
+  while (dot < name_len && name[dot] != '.')
+    ++dot;
+  if (dot == name_len)
+    return FIOBJ_INVALID;
+  tmp = fiobj_mustache_find_obj_tree(section, name, dot);
+  if (!tmp) {
+    return FIOBJ_INVALID;
+  }
+  ++dot;
+  for (;;) {
+    FIOBJ key = fiobj_str_tmp();
+    fiobj_str_write(key, name + dot, name_len - dot);
+    FIOBJ obj = fiobj_mustache_find_obj_absolute(tmp, key);
+    if (obj != FIOBJ_INVALID)
+      return obj;
+    name += dot;
+    name_len -= dot;
+    dot = 0;
+    while (dot < name_len && name[dot] != '.')
+      ++dot;
+    if (dot == name_len) {
+      return FIOBJ_INVALID;
+    }
+    key = fiobj_str_tmp();
+    fiobj_str_write(key, name, dot);
+    tmp = fiobj_mustache_find_obj_absolute(tmp, key);
+    if (tmp == FIOBJ_INVALID)
+      return FIOBJ_INVALID;
+    ++dot;
+  }
+}
+
 /**
  * Called when an argument name was detected in the current section.
  *
@@ -209,11 +257,11 @@ void fiobj_mustache_test(void) {
     exit(-1);                                                                  \
   }
 
-  char const *template = "{{=<< >>=}}* Users:\r\n<<#users>><<id>>. <<& name>> "
-                         "(<<name>>)\r\n<</users>>";
+  char const *template =
+      "{{=<< >>=}}* Users:\r\n<<#users>><<id>>. <<& name>> "
+      "(<<name>>)\r\n<</users>>\r\nNested: <<& nested.item >>.";
   char const *template_name = "mustache_test_template.mustache";
   mustache_save2file(template_name, template, strlen(template));
-  // mustache_error_en err = MUSTACHE_OK;
   mustache_s *m =
       fiobj_mustache_load((fio_str_info_s){.data = (char *)template_name});
   unlink(template_name);
@@ -238,11 +286,19 @@ void fiobj_mustache_test(void) {
     fiobj_free(key);
     fiobj_ary_push(ary, usr);
   }
+  key = fiobj_str_new("nested", 6);
+  ary = fiobj_hash_new2(2);
+  fiobj_hash_set(data, key, ary);
+  fiobj_free(key);
+  key = fiobj_str_new("item", 4);
+  fiobj_hash_set(ary, key, fiobj_str_new("dot notation success", 20));
+  fiobj_free(key);
   key = fiobj_mustache_build(m, data);
   fiobj_free(data);
   TEST_ASSERT(key, "fiobj_mustache_build failed!\n");
   fprintf(stderr, "%s\n", fiobj_obj2cstr(key).data);
   fiobj_free(key);
+  fiobj_mustache_free(m);
 }
 
 #endif

data/ext/iodine/http.c

@@ -1833,7 +1833,7 @@ static void http_mime_parser_on_data(http_mime_parser_s *parser, void *name,
                                      size_t filename_len, void *mimetype,
                                      size_t mimetype_len, void *value,
                                      size_t value_len) {
-  if (!filename) {
+  if (!filename_len) {
     http_add2hash(http_mime_parser2fio(parser)->h->params, name, name_len,
                   value, value_len, 0);
     return;
@@ -1844,15 +1844,17 @@ static void http_mime_parser_on_data(http_mime_parser_s *parser, void *name,
   http_add2hash(http_mime_parser2fio(parser)->h->params, tmp.data, tmp.len,
                 value, value_len, 0);
   fiobj_str_resize(n, name_len);
-  fiobj_str_write(n, "[type]", 6);
-  tmp = fiobj_obj2cstr(n);
-  http_add2hash(http_mime_parser2fio(parser)->h->params, tmp.data, tmp.len,
-                mimetype, mimetype_len, 0);
-  fiobj_str_resize(n, name_len);
   fiobj_str_write(n, "[name]", 6);
   tmp = fiobj_obj2cstr(n);
   http_add2hash(http_mime_parser2fio(parser)->h->params, tmp.data, tmp.len,
                 filename, filename_len, 0);
+  if (mimetype_len) {
+    fiobj_str_resize(n, name_len);
+    fiobj_str_write(n, "[type]", 6);
+    tmp = fiobj_obj2cstr(n);
+    http_add2hash(http_mime_parser2fio(parser)->h->params, tmp.data, tmp.len,
+                  mimetype, mimetype_len, 0);
+  }
   fiobj_free(n);
 }
 
@@ -1900,13 +1902,25 @@ static void http_mime_parser_on_partial_end(http_mime_parser_s *parser) {
 
   fio_str_info_s tmp =
       fiobj_obj2cstr(http_mime_parser2fio(parser)->partial_name);
-  http_add2hash2(http_mime_parser2fio(parser)->h->params, tmp.data, tmp.len,
-                 fiobj_data_slice(http_mime_parser2fio(parser)->h->body,
-                                  http_mime_parser2fio(parser)->partial_offset,
-                                  http_mime_parser2fio(parser)->partial_length),
+  FIOBJ o = FIOBJ_INVALID;
+  if (!http_mime_parser2fio(parser)->partial_length)
+    return;
+  if (http_mime_parser2fio(parser)->partial_length < 42) {
+    /* short data gets a new object */
+    o = fiobj_str_new(http_mime_parser2fio(parser)->buffer.data +
+                          http_mime_parser2fio(parser)->partial_offset,
+                      http_mime_parser2fio(parser)->partial_length);
+  } else {
+    /* longer data gets a reference object (memory collision concerns) */
+    o = fiobj_data_slice(http_mime_parser2fio(parser)->h->body,
+                         http_mime_parser2fio(parser)->partial_offset,
+                         http_mime_parser2fio(parser)->partial_length);
+  }
+  http_add2hash2(http_mime_parser2fio(parser)->h->params, tmp.data, tmp.len, o,
                  0);
   fiobj_free(http_mime_parser2fio(parser)->partial_name);
   http_mime_parser2fio(parser)->partial_name = FIOBJ_INVALID;
+  http_mime_parser2fio(parser)->partial_offset = 0;
 }
 
 /**
@@ -1976,7 +1990,7 @@ int http_parse_body(http_s *h) {
   do {
     size_t cons = http_mime_parse(&p.p, p.buffer.data, p.buffer.len);
     p.pos += cons;
-    p.buffer = fiobj_data_pread(h->body, p.pos, 256);
+    p.buffer = fiobj_data_pread(h->body, p.pos, 4096);
   } while (p.buffer.data && !p.p.done && !p.p.error);
   fiobj_free(p.partial_name);
   p.partial_name = FIOBJ_INVALID;
@@ -2903,5 +2917,6 @@ void http_tests(void) {
   FIOBJ html_mime = http_mimetype_find("html", 4);
   FIO_ASSERT(html_mime,
              "HTML mime-type not found! Mime-Type registry invalid!\n");
+  fiobj_free(html_mime);
 }
 #endif

data/ext/iodine/http1.c

@@ -554,7 +554,7 @@ static int http1_on_request(http1_parser_s *parser) {
   if (p->request.method && !p->stop)
     http_finish(&p->request);
   h1_reset(p);
-  return 0;
+  return fio_is_closed(p->p.uuid);
 }
 /** called when a response was received. */
 static int http1_on_response(http1_parser_s *parser) {
@@ -563,7 +563,7 @@ static int http1_on_response(http1_parser_s *parser) {
   if (p->request.status_str && !p->stop)
     http_finish(&p->request);
   h1_reset(p);
-  return 0;
+  return fio_is_closed(p->p.uuid);
 }
 /** called when a request method is parsed. */
 static int http1_on_method(http1_parser_s *parser, char *method,
@@ -751,7 +751,7 @@ static void http1_on_data_first_time(intptr_t uuid, fio_protocol_s *protocol) {
   /* ensure future reads skip this first time HTTP/2.0 test */
   p->p.protocol.on_data = http1_on_data;
   if (i >= 24 && !memcmp(p->buf, "PRI * HTTP/2.0\r\n\r\nSM\r\n\r\n", 24)) {
-    FIO_LOG_ERROR("unsupported HTTP/2 attempeted using prior knowledge.");
+    FIO_LOG_WARNING("client claimed unsupported HTTP/2 prior knowledge.");
     fio_close(uuid);
     return;
   }

data/ext/iodine/http_mime_parser.h

@@ -10,6 +10,15 @@ Feel free to copy, use and enjoy according to the license provided.
 #include <stdlib.h>
 #include <string.h>
 
+/* *****************************************************************************
+Known Limitations:
+
+- Doesn't support nested multipart form structures (i.e., multi-file selection).
+  See: https://www.w3.org/TR/html401/interact/forms.html#h-17.13.4.2
+
+To circumvent limitation, initialize a new parser to parse nested multiparts.
+***************************************************************************** */
+
 /* *****************************************************************************
 The HTTP MIME Multipart Form Parser Type
 ***************************************************************************** */

data/ext/iodine/iodine.c

@@ -286,13 +286,15 @@ CLI parser (Ruby's OptParser is more limiting than I knew...)
  *
  * @params [String] desc a String containg the iodine server's description.
  */
-static VALUE iodine_cli_parse(VALUE self, VALUE desc) {
+static VALUE iodine_cli_parse(VALUE self) {
   (void)self;
-  Check_Type(desc, T_STRING);
   VALUE ARGV = rb_get_argv();
   VALUE ret = Qtrue;
   VALUE defaults = iodine_default_args;
-  if (!defaults || !ARGV || TYPE(ARGV) != T_ARRAY || TYPE(defaults) != T_HASH) {
+  VALUE iodine_version = rb_const_get(IodineModule, rb_intern("VERSION"));
+  char desc[1024];
+  if (!defaults || !ARGV || TYPE(ARGV) != T_ARRAY || TYPE(defaults) != T_HASH ||
+      TYPE(iodine_version) != T_STRING || RSTRING_LEN(iodine_version) > 512) {
     FIO_LOG_ERROR("CLI parsing initialization error "
                   "ARGV=%p, Array?(%d), defaults == %p (%d)",
                   (void *)ARGV, (int)(TYPE(ARGV) == T_ARRAY), (void *)defaults,
@@ -300,7 +302,7 @@ static VALUE iodine_cli_parse(VALUE self, VALUE desc) {
     return Qnil;
   }
   /* Copy the Ruby ARGV to a C valid ARGV */
-  int argc = (int)rb_array_len(ARGV) + 1;
+  int argc = (int)RARRAY_LEN(ARGV) + 1;
   if (argc <= 1) {
     FIO_LOG_DEBUG("CLI: No arguments to parse...\n");
     return Qnil;
@@ -324,36 +326,52 @@ static VALUE iodine_cli_parse(VALUE self, VALUE desc) {
     argv[i][s.len] = 0;
   }
   /* Levarage the facil.io CLI library */
+  memcpy(desc, "Iodine's HTTP/WebSocket server version ", 39);
+  memcpy(desc + 39, StringValueCStr(iodine_version),
+         RSTRING_LEN(iodine_version));
+  memcpy(desc + 39 + RSTRING_LEN(iodine_version),
+         "\r\n\r\nUse:\r\n    iodine <options> <filename>\r\n\r\n"
+         "Both <options> and <filename> are optional. i.e.,:\r\n"
+         "    iodine -p 0 -b /tmp/my_unix_sock\r\n"
+         "    iodine -p 8080 path/to/app/conf.ru\r\n"
+         "    iodine -p 8080 -w 4 -t 16\r\n"
+         "    iodine -w -1 -t 4 -r redis://usr:pass@localhost:6379/",
+         263);
+  desc[39 + 263 + RSTRING_LEN(iodine_version)] = 0;
   fio_cli_start(
-      argc, (const char **)argv, 0, -1, StringValueCStr(desc),
-      "\x1B[4mAddress Binding:\x1B[0m", FIO_CLI_TYPE_PRINT,
-      "-bind -b -address address to listen to. defaults any available.",
+      argc, (const char **)argv, 0, -1, desc,
+      "Address Binding:", FIO_CLI_TYPE_PRINT_HEADER,
+      "-bind -b -address address to listen to. defaults to any available.",
       "-port -p port number to listen to. defaults port 3000", FIO_CLI_TYPE_INT,
-      "\n\x1B[4mConcurrency:\x1B[0m", FIO_CLI_TYPE_PRINT,
+      "\t\t\x1B[4mNote\x1B[0m: to bind to a Unix socket, "
+      "set \x1B[1mport\x1B[0m to 0.",
+      FIO_CLI_TYPE_PRINT, "Concurrency:", FIO_CLI_TYPE_PRINT_HEADER,
       "-workers -w number of processes to use.", FIO_CLI_TYPE_INT,
       "-threads -t number of threads per process.", FIO_CLI_TYPE_INT,
-      "\n\x1B[4mHTTP Server:\x1B[0m", FIO_CLI_TYPE_PRINT,
+      "HTTP Server:", FIO_CLI_TYPE_PRINT_HEADER,
       "-public -www public folder, for static file service.",
       "-log -v HTTP request logging.", FIO_CLI_TYPE_BOOL,
-      "-keep-alive -k -tout HTTP keep-alive timeout (0..255). Default: 40s",
+      "-keep-alive -k -tout HTTP keep-alive timeout in seconds (0..255). "
+      "Default: 40s",
       FIO_CLI_TYPE_INT, "-ping websocket ping interval (0..255). Default: 40s",
       FIO_CLI_TYPE_INT,
       "-max-body -maxbd HTTP upload limit in Mega-Bytes. Default: 50Mb",
       FIO_CLI_TYPE_INT,
       "-max-header -maxhd header limit per HTTP request in Kb."
       " Default: 32Kb.",
-      FIO_CLI_TYPE_INT, "\n\x1B[4mWebSocket Server:\x1B[0m", FIO_CLI_TYPE_PRINT,
+      FIO_CLI_TYPE_INT, "WebSocket Server:", FIO_CLI_TYPE_PRINT_HEADER,
       "-max-msg -maxms incoming WebSocket message limit in Kb. "
       "Default: 250Kb",
-      FIO_CLI_TYPE_INT, "\n\x1B[4mConnecting Iodine to Redis:\x1B[0m",
-      FIO_CLI_TYPE_PRINT,
+      FIO_CLI_TYPE_INT,
+      "Connecting Iodine to Redis:", FIO_CLI_TYPE_PRINT_HEADER,
       "-redis -r an optional Redis URL server address. Default: none.",
-      "-redis-ping -rp websocket ping interval (0..255). Default: 5 minutes",
-      FIO_CLI_TYPE_INT, "\n\x1B[4mMisc:\x1B[0m", FIO_CLI_TYPE_PRINT,
+      "-redis-ping -rp websocket ping interval (0..255). Default: 300s",
+      FIO_CLI_TYPE_INT, "Misc:", FIO_CLI_TYPE_PRINT_HEADER,
       "-warmup --preload warm up the application. CAREFUL! with workers.",
       FIO_CLI_TYPE_BOOL,
       "-verbosity -V 0..5 server verbosity level. Default: 4",
       FIO_CLI_TYPE_INT);
+
   /* copy values from CLI library to iodine */
   if (fio_cli_get("-V")) {
     int level = fio_cli_get_i("-V");
@@ -472,7 +490,7 @@ void Init_iodine(void) {
   rb_define_module_function(IodineModule, "worker?", iodine_worker_is, 0);
 
   // register CLI methods
-  rb_define_module_function(IodineCLIModule, "parse", iodine_cli_parse, 1);
+  rb_define_module_function(IodineCLIModule, "parse", iodine_cli_parse, 0);
 
   // initialize Object storage for GC protection
   iodine_storage_init();