Add asdf tool-versions definitions pinning this to Nim 1.6

The previous logic that was intended to find the first non-empty
filename that represented a valid file location was wrong. Replaced with
a simpler filter-based version.

Additionally, if the user provides a specific configuration filename, we
no longer fall back to the defaults if it is not found. Instead we just
err out and inform them that the file the specified was not found.

.tool-versions

@@ -0,0 +1 @@
+nim 1.6.20

private/api.nim

@@ -1,7 +1,7 @@
 ## Personal Time Keeping API Interface
 ## ===================================
 
-import asyncdispatch, base64, bcrypt, cliutils, docopt, jester, json, logging,
+import asyncdispatch, base64, bcrypt, cliutils, docopt, httpcore, jester, json, logging,
   sequtils, strutils, os, tables, times, uuids
 
 import nre except toSeq
@@ -61,10 +61,10 @@ template checkAuth(cfg: PtkApiCfg) =
   var user {.inject.}: PtkUser = PtkUser()
 
   try:
-    if not request.headers.hasKey("Authorization"):
+    if not headers(request).hasKey("Authorization"):
       raiseEx "No auth token."
 
-    let headerVal = request.headers["Authorization"]
+    let headerVal = headers(request)["Authorization"]
     if not headerVal.startsWith("Basic "):
       raiseEx "Invalid Authorization type (only 'Basic' is supported)."
 

private/models.nim

@@ -22,10 +22,18 @@ const ISO_TIME_FORMAT* = "yyyy-MM-dd'T'HH:mm:ss"
   ## The canonical time format used by PTK.
 
 const TIME_FORMATS* = @[
+    (fmtStr: "yyyy-MM-dd'T'HH:mm:sszzz", offsetFrom: OffsetFrom.None),
+    (fmtStr: "yyyy-MM-dd HH:mm:sszzz", offsetFrom: OffsetFrom.None),
+    (fmtStr: "yyyy-MM-dd'T'HH:mm:sszz", offsetFrom: OffsetFrom.None),
+    (fmtStr: "yyyy-MM-dd HH:mm:sszz", offsetFrom: OffsetFrom.None),
+    (fmtStr: "yyyy-MM-dd'T'HH:mm:ssz", offsetFrom: OffsetFrom.None),
+    (fmtStr: "yyyy-MM-dd HH:mm:ssz", offsetFrom: OffsetFrom.None),
     (fmtStr: "yyyy-MM-dd'T'HH:mm:ss", offsetFrom: OffsetFrom.None),
     (fmtStr: "yyyy-MM-dd HH:mm:ss", offsetFrom: OffsetFrom.None),
     (fmtStr: "yyyy-MM-dd'T'HH:mm", offsetFrom: OffsetFrom.None),
     (fmtStr: "yyyy-MM-dd HH:mm", offsetFrom: OffsetFrom.None),
+    (fmtStr: "yyyy-MM-dd", offsetFrom: OffsetFrom.None),
+    (fmtStr: "yyyy-MM", offsetFrom: OffsetFrom.None),
     (fmtStr: "MM-dd'T'HH:mm:ss", offsetFrom: OffsetFrom.Year),
     (fmtStr: "MM-dd HH:mm:ss", offsetFrom: OffsetFrom.Year),
     (fmtStr: "MM-dd'T'HH:mm", offsetFrom: OffsetFrom.Year),
@@ -48,15 +56,16 @@ proc getIfExists*(n: JsonNode, key: string): JsonNode =
 
 proc parseTime*(timeStr: string): DateTime =
   ## Helper to parse time strings trying multiple known formats.
+  let now = now()
+
   for fmt in TIME_FORMATS:
     try:
-      let now = now()
       let parsed = parse(timeStr, fmt.fmtStr)
       case fmt.offsetFrom:
         of OffsetFrom.None:
           return parsed
         of OffsetFrom.Year:
-          return initDateTime(parsed.monthday, parsed.month, now.year,
+          return dateTime(now.year, parsed.month, parsed.monthday,
             parsed.hour, parsed.minute, parsed.second, parsed.nanosecond,
             now.timezone)
         of OffsetFrom.Month:

private/version.nim

@@ -1 +1 @@
-const PTK_VERSION* = "1.0.10"
+const PTK_VERSION* = "1.0.14"

ptk.nim

@@ -307,27 +307,21 @@ Options:
     quit()
 
   # Find and parse the .ptkrc file
-  let ptkrcLocations = @[
-    if args["--config"]: $args["--config"] else:"",
-    ".ptkrc", $getEnv("PTKRC"), $getEnv("HOME") & "/.ptkrc"]
+  let ptkrcLocations =
+    if args["--config"]: @[$args["--config"]]
+    else: @[".ptkrc", $getEnv("PTKRC"), $getEnv("HOME") & "/.ptkrc"]
 
-  var ptkrcFilename: string =
-    foldl(ptkrcLocations, if len(a) > 0: a elif fileExists(b): b else: "")
+  let foundPtkrcLocations =
+    ptkrcLocations.filterIt(it.len > 0 and fileExists(it))
 
   var cfg: JsonNode
-  var cfgFile: File
-  if not fileExists(ptkrcFilename):
+  if foundPtkrcLocations.len < 1:
     warn "ptk: could not find .ptkrc file."
-    ptkrcFilename = $getEnv("HOME") & "/.ptkrc"
-    try:
-      cfgFile = open(ptkrcFilename, fmWrite)
-      cfgFile.write("{\"timelineLogFile\": \"timeline.log.json\"}")
-    except: warn "ptk: could not write default .ptkrc to " & ptkrcFilename
-    finally: close(cfgFile)
+    debug "ptk: considered the following locations:\n\t" & ptkrcLocations.join("\n\t")
 
-  try: cfg = parseFile(ptkrcFilename)
+  try: cfg = parseFile(foundPtkrcLocations[0])
   except: raise newException(IOError,
-    "unable to read config file: " & ptkrcFilename &
+    "unable to read config file: " & foundPtkrcLocations[0] &
     "\x0D\x0A" & getCurrentExceptionMsg())
 
   # Find the time log file

ptk.nimble

@@ -1,6 +1,6 @@
 # Package
 
-version       = "1.0.10"
+version       = "1.0.14"
 author        = "Jonathan Bernard"
 description   = "Personal Time Keeper"
 license       = "MIT"