Ensure absolute URLs for single lines from Banzai for HipChat

"pipeline: :single_line" leaves the protocol/host part out of the URLs
and caches them that way. To avoid giving those out to HipChat, markdown
is always rendered with "pipeline: :email" first.

There must be a better way to do this, but I can't see how to avoid the
link caching.

1 files changed, 7 insertions, 3 deletions

diff --git a/app/models/project_services/hipchat_service.rb b/app/models/project_services/hipchat_service.rb
index 9d52a1423b7..ce4a2a96015 100644
--- a/app/models/project_services/hipchat_service.rb
+++ b/app/models/project_services/hipchat_service.rb
@@ -125,12 +125,16 @@ class HipchatService < Service
     message
   end
 
-  def markdown(text, context = {})
+  def markdown(text, options = {})
     if text
-      context = ({
+      context = {
         project: project,
         pipeline: :email
-      }).merge(context)
+      }
+
+      Banzai.render(text, context)
+
+      context.merge!(options)
 
       html = Banzai.render(text, context)
       html = Banzai.post_process(html, context)