Last modified by Agnease on 2026/06/16 17:18

From version 16.50
edited by Agnease
on 2026/06/16 17:02
Change comment: There is no comment for this version
To version 16.57
edited by Agnease
on 2026/06/16 17:18
Change comment: There is no comment for this version

Summary

Details

Page properties
Content
... ... @@ -3,50 +3,113 @@
3 3   #set ($statusCode = 400)
4 4   #set ($message = 'The request could not be sent. Please try again or contact Agnease by email at alex@agnease.com.')
5 5  
6 + #set ($className = 'Agnease.Code.ContactRequest.ContactRequestClass')
7 + #set ($allowedProperties = [
8 + 'scope',
9 + 'alreadyUseXWiki',
10 + 'name',
11 + 'email',
12 + 'hosting',
13 + 'customDevelopment',
14 + 'timeline',
15 + 'users'
16 + ])
17 +
6 6   #set ($name = '')
7 7   #set ($email = '')
8 - ## Fields to help preventing bots filled forms.
20 + #set ($scope = '')
9 9   #set ($contactWebsite = '')
10 10   #set ($startedAtRaw = '')
11 11  
24 + ## Extract only the values we need for validation.
12 12   #foreach ($parameterName in $request.parameterNames)
13 13   #set ($propertyParts = $parameterName.split('_0_'))
14 14   #if ($propertyParts.size() > 1)
15 - #set ($propertyName = $parameterName.split('_0_')[1])
28 + #set ($propertyName = $propertyParts[1])
29 + #set ($propertyValue = $stringtool.trim($request.get($parameterName)))
30 +
16 16   #if ($propertyName == 'name')
17 - #set ($name = $stringtool.trim($request.get($parameterName)))
32 + #set ($name = $propertyValue)
18 18   #elseif ($propertyName == 'email')
19 - #set ($email = $stringtool.trim($request.get($parameterName)))
34 + #set ($email = $propertyValue)
35 + #elseif ($propertyName == 'scope')
36 + #set ($scope = $propertyValue)
20 20   #elseif ($propertyName == 'contactWebsite')
21 - #set ($contactWebsite = $stringtool.trim($request.get($parameterName)))
38 + #set ($contactWebsite = $propertyValue)
22 22   #elseif ($propertyName == 'contactStartedAt')
23 - #set ($startedAtRaw = $stringtool.trim($request.get($parameterName)))
40 + #set ($startedAtRaw = $propertyValue)
24 24   #end
25 25   #end
26 26   #end
27 27  
28 - #if ("$!startedAtRaw" != '')
45 + #set ($spamScore = 0)
46 +
47 + ## Honeypot: real users should never fill this field.
48 + #if ("$!contactWebsite" != '')
49 + #set ($spamScore = $spamScore + 5)
50 + #end
51 +
52 + ## Submission timing check.
53 + #if ("$!startedAtRaw" == '')
54 + ## The field is expected from the real form, so missing it is suspicious.
55 + #set ($spamScore = $spamScore + 2)
56 + #else
29 29   #set ($startedAt = $numbertool.toNumber($startedAtRaw))
30 - #set ($now = $datetool.systemDate.time)
31 - #set ($elapsed = $now - $startedAt)
58 + #if ("$!startedAt" == '')
59 + #set ($spamScore = $spamScore + 2)
60 + #else
61 + #set ($now = $datetool.systemDate.time)
62 + #set ($elapsed = $now - $startedAt)
32 32  
33 - ## Reject submissions faster than 10 seconds.
34 - #if ($elapsed > 0 && $elapsed < 10000)
35 - #set ($statusCode = 400)
36 - #set ($message = 'Please take a moment to describe your XWiki request before submitting.')
64 + ## Reject very fast submissions.
65 + #if ($elapsed > 0 && $elapsed < 10000)
66 + #set ($spamScore = $spamScore + 3)
67 + #end
37 37   #end
38 - #elseif ("$!contactWebsite.trim()" != '')
39 - #set ($statusCode = 400)
40 - #set ($message = 'The request could not be sent. Please try again or contact Agnease by email.')
41 - #elseif ("$!name" == '' && "$!email" == '')
42 - #set ($statusCode = 400)
69 + #end
70 +
71 + ## Random-looking name: long single token.
72 + #if ($name.length() >= 16 && !$name.contains(' '))
73 + #set ($spamScore = $spamScore + 2)
74 + #end
75 +
76 + ## Random-looking project description: long single token.
77 + #if ($scope.length() >= 12 && !$scope.contains(' '))
78 + #set ($spamScore = $spamScore + 2)
79 + #end
80 +
81 + ## Suspicious email local part with many dots and tiny fragments.
82 + #set ($emailParts = $email.split('@'))
83 + #if ($emailParts.size() == 2)
84 + #set ($localPart = $emailParts[0])
85 + #set ($localFragments = $localPart.split('\.'))
86 + #set ($dotCount = $localFragments.size() - 1)
87 + #set ($oneCharFragments = 0)
88 +
89 + #foreach ($fragment in $localFragments)
90 + #if ($fragment.length() == 1)
91 + #set ($oneCharFragments = $oneCharFragments + 1)
92 + #end
93 + #end
94 +
95 + #if ($dotCount >= 4 && $oneCharFragments >= 3)
96 + #set ($spamScore = $spamScore + 2)
97 + #end
98 + #else
99 + #set ($spamScore = $spamScore + 2)
100 + #end
101 +
102 + ## Human-facing validation.
103 + #if ("$!name" == '' && "$!email" == '')
43 43   #set ($message = 'Please enter your name and email.')
44 44   #elseif ("$!name" == '')
45 - #set ($statusCode = 400)
46 46   #set ($message = 'Please enter your name.')
47 47   #elseif ("$!email" == '')
48 - #set ($statusCode = 400)
49 49   #set ($message = 'Please enter your email address.')
109 + #elseif ("$!scope" == '' || $scope.length() < 30)
110 + #set ($message = 'Please add a short description of your XWiki project, question or issue.')
111 + #elseif ($spamScore >= 3)
112 + #set ($message = 'The request could not be sent. Please add a clearer description of your XWiki request or contact Agnease by email.')
50 50   #else
51 51   #try('contactException')
52 52   #set ($now = $datetool.get('yyyyMMddHHmm'))
... ... @@ -53,11 +53,18 @@
53 53   #set ($random = $mathtool.random(100000, 999999))
54 54   #set ($uniqueName = "ContactRequest-${now}-${random}")
55 55   #set ($contactRequestDoc = $xwiki.getDocumentAsAuthor('ContactRequests.' + $uniqueName))
56 - #set ($contactRequestObj = $contactRequestDoc.getObject('Agnease.Code.ContactRequest.ContactRequestClass', true))
119 + #set ($contactRequestObj = $contactRequestDoc.getObject($className, true))
57 57  
121 + ## Save only known ContactRequest fields.
58 58   #foreach ($parameterName in $request.parameterNames)
59 - #set ($propertyName = $parameterName.split('_0_')[1])
60 - #set ($discard = $contactRequestObj.set($propertyName, $request.get($parameterName)))
123 + #set ($propertyParts = $parameterName.split('_0_'))
124 + #if ($propertyParts.size() > 1)
125 + #set ($propertyName = $propertyParts[1])
126 +
127 + #if ($allowedProperties.contains($propertyName))
128 + #set ($discard = $contactRequestObj.set($propertyName, $request.get($parameterName)))
129 + #end
130 + #end
61 61   #end
62 62  
63 63   #set ($discard = $contactRequestDoc.saveAsAuthor())