Skip to content

Commit 69e9caa

Browse files
committed
jsext: move field wrapper config to common VM setup routine
1 parent 117ac00 commit 69e9caa

File tree

3 files changed

+1
-2
lines changed

3 files changed

+1
-2
lines changed

access/jsfilter.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -46,7 +46,6 @@ func NewJSFilter(filename string, instances int, logger *clog.CondLogger, next F
4646
if err != nil {
4747
return fmt.Errorf("can't configure runtime: %w", err)
4848
}
49-
vm.SetFieldNameMapper(goja.TagFieldNameMapper("json", true))
5049
_, err = vm.RunString(string(script))
5150
if err != nil {
5251
return fmt.Errorf("script run failed: %w", err)

dialer/jsrouter.go

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,6 @@ func NewJSRouter(filename string, instances int, factory func(string) (Dialer, e
4444
if err != nil {
4545
return fmt.Errorf("can't configure runtime runtime: %w", err)
4646
}
47-
vm.SetFieldNameMapper(goja.TagFieldNameMapper("json", true))
4847
_, err = vm.RunString(string(script))
4948
if err != nil {
5049
return fmt.Errorf("script run failed: %w", err)

jsext/jsext.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,7 @@ package jsext
33
import "github.com/dop251/goja"
44

55
func ConfigureRuntime(vm *goja.Runtime) error {
6+
vm.SetFieldNameMapper(goja.TagFieldNameMapper("json", true))
67
if err := AddFileReader(vm); err != nil {
78
return err
89
}

0 commit comments

Comments
 (0)