nadesiko3 3.3.74 → 3.3.76

package/src/cnako3.mts

@@ -1,5 +1,4 @@
 #!/usr/bin/env node
-
 /**
  * コマンドライン版のなでしこ3
  */

package/src/nako_version.mjs

@@ -1,8 +1,8 @@
 // 実際のバージョン定義 (自動生成されるので以下を編集しない)
 const nakoVersion = {
-    version: '3.3.74',
+    version: '3.3.76',
     major: 3,
     minor: 3,
-    patch: 74
+    patch: 76
 };
 export default nakoVersion;

package/src/nako_version.mts

@@ -11,9 +11,9 @@ export interface NakoVersion {
 }
 // 実際のバージョン定義 (自動生成されるので以下を編集しない)
 const nakoVersion: NakoVersion = {
-  version: '3.3.74',
+  version: '3.3.76',
   major: 3,
   minor: 3,
-  patch: 74
+  patch: 76
 }
 export default nakoVersion

package/tools/nako3edit/html/edit.html

@@ -14,6 +14,7 @@
 #-----------
 APPKEY=""
 HREF=WINDOW["location"]["href"]
+HREFを表示
 P=HREFをURLパラメータ解析
 APPKEY=P["appkey"]
 Pを表示。

package/tools/nako3edit/index.mjs

@@ -1,7 +1,7 @@
 #!/usr/bin/env node
 /** nako3edit用の超簡易サーバ(生のnodeだけで簡単HTTPサーバ) */
 import path from 'path'
-import fs, { existsSync } from 'fs'
+import fs from 'fs'
 import { execSync } from 'child_process'
 import opener from 'opener'
 import http from 'http'
@@ -20,7 +20,11 @@ const homeDir = process.env[isWin ? 'USERPROFILE' : 'HOME']
 const userDir = path.join(homeDir, 'nadesiko3_user')
 const CNAKO3 = path.resolve(path.join(__dirname, '../../src/cnako3.mjs'))
 const NODE = process.argv[0]
-const appkey = 'K' + Math.floor(Math.random() * 10000000).toString(16)
+const appkey = 'k' +
+  Math.floor(Math.random() * 0xFFFFFFFF).toString(16) +
+  Math.floor(Math.random() * 0xFFFFFFFF).toString(16) +
+  Math.floor(Math.random() * 0xFFFFFFFF).toString(16) +
+  Math.floor(Math.random() * 0xFFFFFFFF).toString(16)
 
 // ユーザーフォルダを作成
 if (!fs.existsSync(userDir)) { fs.mkdirSync(userDir) }
@@ -44,7 +48,11 @@ const server = http.createServer(function (req, res) {
     const q = String(a[1]).split('&')
     for (const kv of q) {
       const qq = kv.split('=')
-      params[qq[0]] = decodeURIComponent(qq[1])
+      try {
+        params[qq[0]] = decodeURIComponent(qq[1])
+      } catch (e) {
+        console.error(e)
+      }
     }
   }
   // サニタイズ
@@ -155,7 +163,7 @@ function apiFiles (res) {
   res.end(JSON.stringify(files))
 }
 function apiLoad (res, params) {
-  const fname = params.file
+  const fname = removePathFlag(params.file)
   const fullpath = path.join(userDir, fname)
   console.log('load=', fullpath)
   let text = '# 新規ファイル\n「こんにちは」と表示。'
@@ -172,7 +180,7 @@ function apiSave (res, params) {
     res.end('[ERROR] キーが違います')
     return
   }
-  const fname = params.file
+  const fname = removePathFlag(params.file)
   const body = params.body
   const fullpath = path.join(userDir, fname)
   try {
@@ -186,6 +194,13 @@ function apiSave (res, params) {
   }
 }
 
+function removePathFlag (s) {
+  // ファイル名をサニタイズ
+  s = s.replace(/['"`\\?/<>*]/g, '_')
+  s = s.replace(/_{2,}/g, '') // '__'を削除
+  return s
+}
+
 function apiRun (res, params) {
   res.writeHead(200, { 'Content-Type': 'text/plain; charset=utf-8' })
   const appkeyUser = params.appkey
@@ -193,7 +208,7 @@ function apiRun (res, params) {
     res.end('[ERROR] キーが違います')
     return
   }
-  const fname = params.file
+  const fname = removePathFlag(params.file)
   const body = params.body
   const fullpath = path.join(userDir, fname)
   try {
@@ -205,6 +220,8 @@ function apiRun (res, params) {
       result = String(result)
     } catch (err) {
       console.error(err)
+      res.end('[ERROR]実行に失敗しました。' + err.toString())
+      return
     }
     console.log('[run] file=', fname)
     console.log('--------------------------------')
@@ -212,6 +229,7 @@ function apiRun (res, params) {
     console.log('--------------------------------')
     res.end(result)
   } catch (err) {
+    console.error(err)
     res.end('[ERROR] 実行に失敗しました。')
   }
 }
@@ -220,27 +238,31 @@ function apiDelete (res, params) {
   res.writeHead(200, { 'Content-Type': 'text/plain; charset=utf-8' })
   const appkeyUser = params.appkey
   if (appkey !== appkeyUser) {
-    res.end('[ERROR] キーが違います')
+    res.end('"[ERROR] キーが違います"')
     return
   }
-  const fname = params.file
-  const body = params.body
+  const fname = removePathFlag(params.file)
   const fullpath = path.join(userDir, fname)
   try {
-    fs.unlinkSync(fullpath)
-    res.end('"ok"')
+    if (fs.existsSync(fullpath)) {
+      fs.unlinkSync(fullpath)
+      res.end('"ok"')
+    } else {
+      res.end('"[ERROR] ファイルが見つかりません。"')
+    }
     return
   } catch (err) {
+    console.error(err)
     res.end('error:' + err.message)
   }
 }
 
 function apiGetNewFilename (res) {
   let fname = 'newfile.nako3'
-  for (let i = 1; i <= 999; i++) {
-    fname = `newfile${i}.nako3`
+  for (let i = 1; i <= 9999; i++) {
+    fname = `file${i}.nako3`
     const full = path.join(userDir, fname)
-    if (fs.existsSync(fname)) { continue }
+    if (fs.existsSync(full)) { continue }
     break
   }
   res.writeHead(200, { 'Content-Type': 'text/plaing; charset=utf-8' })