Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

Loading…

job url should not contain "//" #1

Merged
merged 1 commit into from

2 participants

@masaki
  • checking root_url trailing slash
@kentaro kentaro merged commit 1fb4b45 into kentaro:master
@kentaro
Owner

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Nov 8, 2012
  1. @masaki

    check trailing slash

    masaki authored
This page is out of date. Refresh to see the latest.
Showing with 3 additions and 1 deletion.
  1. +3 −1 lib/ikachan/message.rb
View
4 lib/ikachan/message.rb
@@ -11,7 +11,9 @@ def message
instance = Java::jenkins::model::Jenkins.instance
root_url = (instance && instance.root_url) || ''
- msg << " (#{root_url}/#{build.url})"
+
+ url = [ root_url, build.url ].join(root_url[-1] == '/' ? '' : '/')
+ msg << " (#{url})"
"\x03%s%s" % [code.to_s, msg]
end
Something went wrong with that request. Please try again.