ZReviewTender 0.0.4 → 0.0.7
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- checksums.yaml +4 -4
- data/lib/Processors/SlackProcessor.rb +26 -5
- metadata +1 -1
checksums.yaml
CHANGED
|
@@ -1,7 +1,7 @@
|
|
|
1
1
|
---
|
|
2
2
|
SHA256:
|
|
3
|
-
metadata.gz:
|
|
4
|
-
data.tar.gz:
|
|
3
|
+
metadata.gz: 556b1175c3cb37d22d19f7ad2f8efdcd029cebb30cee4dcec2f98bff1f6b9a5b
|
|
4
|
+
data.tar.gz: 17c67831cb8bbafb89c51c1c551f778a18f8d5f595ec5ed6fcff3c029657fd0c
|
|
5
5
|
SHA512:
|
|
6
|
-
metadata.gz:
|
|
7
|
-
data.tar.gz:
|
|
6
|
+
metadata.gz: a42d994b153e45fae773118eb29320339d2e9c49d52d865dd60bb62fb23c0aa5c566386cbfeebd0608bc56ab6b5affcf4782de89451f29b7dcb61276208cfc37
|
|
7
|
+
data.tar.gz: 1d1b1186d77d05e9de46dfeceb38adb9d538a297c3e2a1c0803099b9266ed414c2e9b3854eaf44ec49c3685ad4d03f6222622a18b94e764a71e8cfa79a88dd12
|
|
@@ -9,7 +9,7 @@ require "time"
|
|
|
9
9
|
|
|
10
10
|
class SlackProcessor < Processor
|
|
11
11
|
|
|
12
|
-
attr_accessor :botToken, :inCommingWebHookURL, :targetChannel, :timeZoneOffset
|
|
12
|
+
attr_accessor :botToken, :inCommingWebHookURL, :targetChannel, :timeZoneOffset, :attachmentGroupByNumber
|
|
13
13
|
|
|
14
14
|
def initialize(config, configFilePath, baseExecutePath)
|
|
15
15
|
@config = config
|
|
@@ -20,6 +20,11 @@ class SlackProcessor < Processor
|
|
|
20
20
|
@inCommingWebHookURL = config["slackInCommingWebHookURL"]
|
|
21
21
|
@targetChannel = config["slackBotTargetChannel"]
|
|
22
22
|
@timeZoneOffset = Helper.unwrapRequiredParameter(config, "slackTimeZoneOffset")
|
|
23
|
+
@attachmentGroupByNumber = 1
|
|
24
|
+
|
|
25
|
+
if !config['slackAttachmentGroupByNumber'].nil? && config['slackAttachmentGroupByNumber'] != "" && config['slackAttachmentGroupByNumber'].to_i > 0 && config['slackAttachmentGroupByNumber'].to_i < 100
|
|
26
|
+
@attachmentGroupByNumber = config['slackAttachmentGroupByNumber'].to_i
|
|
27
|
+
end
|
|
23
28
|
|
|
24
29
|
if (botToken.nil? && inCommingWebHookURL.nil?) || (botToken == "" && inCommingWebHookURL == "")
|
|
25
30
|
raise "must specify slackBotToken or slackInCommingWebHookURL in SlackProcessor."
|
|
@@ -29,7 +34,11 @@ class SlackProcessor < Processor
|
|
|
29
34
|
end
|
|
30
35
|
|
|
31
36
|
def processReviews(reviews, platform)
|
|
32
|
-
|
|
37
|
+
|
|
38
|
+
pendingPayloads = []
|
|
39
|
+
|
|
40
|
+
# Slack Message Limit: posting one message per second per channel
|
|
41
|
+
reviews.each_slice(attachmentGroupByNumber) do |reviewGroup|
|
|
33
42
|
payload = Payload.new()
|
|
34
43
|
payload.attachments = []
|
|
35
44
|
|
|
@@ -55,10 +64,22 @@ class SlackProcessor < Processor
|
|
|
55
64
|
payload.attachments.append(attachment)
|
|
56
65
|
end
|
|
57
66
|
|
|
67
|
+
pendingPayloads.append(payload)
|
|
68
|
+
end
|
|
69
|
+
|
|
70
|
+
loop do
|
|
71
|
+
payload = pendingPayloads.shift
|
|
72
|
+
|
|
58
73
|
result = request(payload)
|
|
59
|
-
if !result
|
|
74
|
+
if !result[:ok]
|
|
60
75
|
Helper.logError(result)
|
|
76
|
+
if result[:message] == "ratelimited"
|
|
77
|
+
sleep(1)
|
|
78
|
+
pendingPayloads.insert(0, payload)
|
|
79
|
+
end
|
|
61
80
|
end
|
|
81
|
+
|
|
82
|
+
break if pendingPayloads.length < 1
|
|
62
83
|
end
|
|
63
84
|
|
|
64
85
|
return reviews
|
|
@@ -108,10 +129,10 @@ class SlackProcessor < Processor
|
|
|
108
129
|
res = http.request(req)
|
|
109
130
|
|
|
110
131
|
if isInCommingWebHook
|
|
111
|
-
return res.body == "ok"
|
|
132
|
+
return {"ok":res.body == "ok", "message":nil}
|
|
112
133
|
else
|
|
113
134
|
result = JSON.parse(res.body)
|
|
114
|
-
return result["ok"] == true
|
|
135
|
+
return {"ok":result["ok"] == true, "message":result['error']}
|
|
115
136
|
end
|
|
116
137
|
|
|
117
138
|
end
|