DeforaNetworks/corteza
3
0
Fork 0
Code Activity

Re-sanitize values on record update/create triggers

Browse Source 
Fixes #30
This commit is contained in:
Denis Arh
2019-10-09 08:03:39 +02:00
parent 982de0d1d9
commit f1b4c3a842
1 changed files with 13 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files
compose/service/record.go
+13 -1
View File
@@ -327,6 +327,12 @@ func (svc record) Create(mod *types.Record) (r *types.Record, err error) {
return
}
// We do not know what happened in the before-create script,
// so we must sanitize values again before we store it
if r.Values, err = svc.sanitizeValues(m, r.Values); err != nil {
return
}
defer func() {
// Run this at the end and discard the error
_ = svc.sr.AfterRecordCreate(svc.ctx, ns, m, r)
@@ -372,8 +378,14 @@ func (svc record) Update(mod *types.Record) (r *types.Record, err error) {
mod = nil // make sure we do not use it anymore
// Calling before-record-update scripts
if err = svc.sr.BeforeRecordUpdate(svc.ctx, ns, m, r); err != nil {
// Calling
return
}
// We do not know what happened in the before-update script,
// so we must sanitize values again before we store it
if r.Values, err = svc.sanitizeValues(m, r.Values); err != nil {
return
}