Merged pull request #1228 from sj14/master

check if replay file has an extension
influxdata · Feb 27, 2017 · cfbbd88 · cfbbd88
2 parents d57e7ba + 8e3c429
commit cfbbd88
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -29,7 +29,7 @@
     we decided it would be best to fix the issue now and make it consistent for all future builds.
     Specifically in the JSON result data the old key `Series` is always `series`, and the old key `Err` is now always `error` instead of for only some of the outputs.
 - [#1181](https://github.com/influxdata/kapacitor/pull/1181): Fix bug parsing dbrp values with quotes.
-
+- [#1228](https://github.com/influxdata/kapacitor/pull/1228): Fix panic on loading replay files without a file extension.
 
 ## v1.2.0 [2017-01-23]
 

diff --git a/services/replay/service.go b/services/replay/service.go
@@ -218,6 +218,10 @@ func (s *Service) syncRecordingMetadata() error {
 		}
 		name := info.Name()
 		i := strings.LastIndex(name, ".")
+		if i == -1 {
+			s.logger.Println("E! file without extension in replay dir", name)
+			continue
+		}
 		ext := name[i:]
 		id := name[:i]