openstatus.dev / openstatus
Openstatus www.openstatus.dev
fork atom

Feat/never panic (#737)

* feat: update Dockerfile

* feat: do not panic

---------

Co-authored-by: Thibault Le Ouay <thibaultleouay@gmail.Com>

authored by

Arthur EICHELBERGER
Thibault Le Ouay and committed by
GitHub 631acb75 37665b53

+2 -1
+1 -1
apps/checker/pkg/assertions/assertions.go
··· 63 63 headers := make(map[string]any) 64 64 65 65 if err := json.Unmarshal([]byte(s), &headers); err != nil { 66 - panic(err) 66 + return false 67 67 } 68 68 69 69 v, found := headers[target.Key]
+1
apps/checker/pkg/assertions/assertions_test.go
··· 102 102 }{ 103 103 {name: "Header 1", fields: fields{Comparator: request.StringEmpty, Target: "", Key: "headers1"}, args: args{s: `{"Content-Type":"text/plain;charset=UTF-8","Strict-Transport-Security":"max-age=3153600000","Vary":"Accept-Encoding"}`}, want: false}, 104 104 {name: "Header 2", fields: fields{Comparator: request.StringNotEmpty, Target: "", Key: "headers1"}, args: args{s: `{"Content-Type":"text/plain;charset=UTF-8","Strict-Transport-Security":"max-age=3153600000","headers1":"Accept-Encoding"}`}, want: true}, 105 + {name: "it should return false if it can not decode the headers", fields: fields{Comparator: request.StringContains, Target: "Accept-Encoding", Key: "Vary"}, args: args{s: `}`}, want: false}, 105 106 } 106 107 for _, tt := range tests { 107 108 t.Run(tt.name, func(t *testing.T) {