Skip to content

Commit 04457b0

Browse files
author
root
committed
Fix #<TypeError: no implicit conversion of nil into String> for post receive urls too
1 parent f9cd767 commit 04457b0

File tree

1 file changed

+14
-10
lines changed

1 file changed

+14
-10
lines changed

Diff for: app/services/hooks/webservices.rb

+14-10
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,18 @@ def execute(repository, payloads)
4747

4848

4949
def execute
50-
call_webservice if needs_push?
50+
y = ''
51+
52+
logger.info("Notifying #{url} ... ")
53+
y << " - Notifying #{url} ... "
54+
55+
if needs_push?
56+
y << call_webservice
57+
else
58+
y << "This url doesn't need to be notified\n"
59+
end
60+
61+
y
5162
end
5263

5364

@@ -110,23 +121,16 @@ def call_webservice
110121

111122

112123
def do_call_webservice(payload)
113-
y = ''
114-
115-
logger.info("Notifying #{url} ... ")
116-
y << " - Notifying #{url} ... "
117-
118124
post_failed, post_message = self.send(use_method, url, {data: { payload: payload }})
119125

120126
if post_failed
121127
logger.error('Failed!')
122128
logger.error("#{post_message}")
123-
y << " [failure]\n"
129+
" [failure]\n"
124130
else
125131
logger.info('Succeeded!')
126-
y << " [success]\n"
132+
" [success]\n"
127133
end
128-
129-
y
130134
end
131135

132136

0 commit comments

Comments
 (0)