Ensure argv is passed through properly

Author: Qard

README.md

@@ -58,6 +58,7 @@ console.log(response.body.toString())
 ### `new Php(config)`
 
 * `config` {Object} Configuration object
+  * `argv` {String[]} Process arguments. **Default:** []
   * `docroot` {String} Document root for PHP. **Default:** process.cwd()
 * Returns: {Php}
 
@@ -67,6 +68,7 @@ Construct a new PHP instance to which to dispatch requests.
 import { Php } from '@platformatic/php-node'
 
 const php = new Php({
+  argv: process.argv,
   docroot: process.cwd()
 })
 ````

__test__/handler.spec.mjs

@@ -15,12 +15,11 @@ test('Support input/output streams', async (t) => {
   t.teardown(() => mockroot.clean())
 
   const php = new Php({
-    argv: process.argv,
     docroot: mockroot.path
   })
 
   const req = new Request({
-    method: 'GET',
+    method: 'POST',
     url: 'http://example.com/index.php',
     body: Buffer.from('Hello, from Node.js!')
   })
@@ -39,12 +38,10 @@ test('Capture logs', async (t) => {
   t.teardown(() => mockroot.clean())
 
   const php = new Php({
-    argv: process.argv,
     docroot: mockroot.path
   })
 
   const req = new Request({
-    method: 'GET',
     url: 'http://example.com/index.php'
   })
 
@@ -62,12 +59,10 @@ test('Capture exceptions', async (t) => {
   t.teardown(() => mockroot.clean())
 
   const php = new Php({
-    argv: process.argv,
     docroot: mockroot.path
   })
 
   const req = new Request({
-    method: 'GET',
     url: 'http://example.com/index.php'
   })
 
@@ -89,12 +84,10 @@ test('Support request and response headers', async (t) => {
   t.teardown(() => mockroot.clean())
 
   const php = new Php({
-    argv: process.argv,
     docroot: mockroot.path
   })
 
   const req = new Request({
-    method: 'GET',
     url: 'http://example.com/index.php',
     headers: {
       'X-Test': ['Hello, from Node.js!']
@@ -106,3 +99,33 @@ test('Support request and response headers', async (t) => {
   t.is(res.body.toString(), 'Hello, from Node.js!')
   t.is(res.headers.get('X-Test'), 'Hello, from PHP!')
 })
+
+test('Has expected args', async (t) => {
+  const mockroot = await MockRoot.from({
+    'index.php': `<?php
+      echo "[";
+      $first = true;
+      foreach ($argv as $value) {
+        if ($first) { $first = false; }
+        else { echo ","; }
+        echo "\\"$value\\"";
+      }
+      echo "]";
+    ?>`
+  })
+  t.teardown(() => mockroot.clean())
+
+  const php = new Php({
+    argv: process.argv,
+    docroot: mockroot.path
+  })
+
+  const req = new Request({
+    url: 'http://example.com/index.php'
+  })
+
+  const res = await php.handleRequest(req)
+  t.is(res.status, 200)
+
+  t.is(res.body.toString('utf8'), JSON.stringify(process.argv))
+})

crates/php/src/embed.rs

@@ -1,6 +1,6 @@
 use std::{
   env::Args,
-  ffi::c_char,
+  ffi::{c_char, CString},
   path::{Path, PathBuf},
   sync::Arc,
 };
@@ -189,6 +189,15 @@ impl Handler for Embed {
       .unwrap_or(0);
     let cookie_data = nullable_cstr(headers.get("Cookie"))?;
 
+    let argc = self.args.len() as i32;
+    let mut argv_ptrs = vec![];
+    for arg in self.args.iter() {
+      let string = CString::new(arg.as_bytes()).map_err(|_| {
+        EmbedException::CStringEncodeFailed(arg.to_owned())
+      })?;
+      argv_ptrs.push(string.into_raw());
+    }
+
     // Prepare memory stream of the code
     let mut file_handle = unsafe {
       let mut file_handle = zend_file_handle {
@@ -221,8 +230,8 @@ impl Handler for Embed {
 
         // Reset state
         globals.request_info.proto_num = 110;
-        globals.request_info.argc = 0;
-        globals.request_info.argv = std::ptr::null_mut();
+        globals.request_info.argc = argc;
+        globals.request_info.argv = argv_ptrs.as_mut_ptr();
         globals.request_info.headers_read = false;
         globals.sapi_headers.http_response_code = 200;
 

crates/php/src/sapi.rs

@@ -201,6 +201,10 @@ pub extern "C" fn sapi_module_deactivate() -> c_int {
   {
     let mut globals = SapiGlobals::get_mut();
 
+    for i in 0..globals.request_info.argc {
+      drop_str(unsafe { *globals.request_info.argv.offset(i as isize) });
+    }
+
     globals.server_context = std::ptr::null_mut();
     globals.request_info.argc = 0;
     globals.request_info.argv = std::ptr::null_mut();