Move path fix to broadcast_message model

app/models/broadcast_message.rb

@@ -106,6 +106,14 @@ class BroadcastMessage < ApplicationRecord
     return false if current_path.blank? && target_path.present?
     return true if current_path.blank? || target_path.blank?
 
+    # Ensure paths are consistent across callers.
+    # This fixes a mismatch between requests in the GUI and CLI
+    #
+    # This has to be reassigned due to frozen strings being provided.
+    unless current_path.start_with?("/")
+      current_path = "/#{current_path}"
+    end
+
     escaped = Regexp.escape(target_path).gsub('\\*', '.*')
     regexp = Regexp.new "^#{escaped}$", Regexp::IGNORECASE
 

app/services/post_receive_service.rb

@@ -81,12 +81,8 @@ class PostReceiveService
     banner = nil
 
     if project
-      # This provides the path in a similar manner to the frontend passing
-      # the request path, improving regex matching.
-      path = "/#{project.full_path}"
-
-      scoped_messages = BroadcastMessage.current_banner_messages(path).select do |message|
-        message.target_path.present? && message.matches_current_path(path)
+      scoped_messages = BroadcastMessage.current_banner_messages(project.full_path).select do |message|
+        message.target_path.present? && message.matches_current_path(project.full_path)
       end
 
       banner = scoped_messages.last