Solr Flood Control message appears publically

Created on 27 June 2024, 7 months ago
Updated 20 September 2024, 4 months ago

Problem/Motivation

Solr Flood Control message appears publicly which does not provide a useful experience for site visitors

Proposed resolution

The message can be hidden or logged in to the backend

📌 Task
Status

Fixed

Version

3.1

Component

Code

Created by

🇮🇳India vipin.mittal18 Greater Noida

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @vipin.mittal18
  • Pipeline finished with Failed
    7 months ago
    Total: 360s
    #209287
  • 🇮🇳India vipin.mittal18 Greater Noida
  • Pipeline finished with Failed
    7 months ago
    Total: 516s
    #209305
  • Pipeline finished with Failed
    7 months ago
    Total: 621s
    #209428
  • First commit to issue fork.
  • Pipeline finished with Failed
    7 months ago
    Total: 227s
    #214597
  • Pipeline finished with Failed
    7 months ago
    Total: 268s
    #214598
  • Pipeline finished with Failed
    7 months ago
    #214618
  • Status changed to Needs review 7 months ago
  • Status changed to Needs work 7 months ago
  • 🇮🇳India vipin.mittal18 Greater Noida

    Need to define RfcLogLevel

  • Pipeline finished with Failed
    7 months ago
    Total: 265s
    #214696
  • Status changed to Needs review 7 months ago
  • 🇮🇳India vipin.mittal18 Greater Noida
  • Issue was unassigned.
  • 🇮🇳India vipin.mittal18 Greater Noida
  • 🇮🇳India vipin.mittal18 Greater Noida

    Attached patch till these changes included in next version release.

  • Pipeline finished with Success
    7 months ago
    Total: 280s
    #214759
  • Pipeline finished with Skipped
    4 months ago
    #285862
  • First commit to issue fork.
  • Status changed to Fixed 4 months ago
  • 🇺🇸United States japerry KVUO
  • 🇮🇳India deepakkm

    Hello @japerry - Are there any reasons why credit is not given for this issue, considering that the merged commit was made by me?

    • japerry committed 4e3a3b68 on 3.2.x
      Issue #3457490: Solr Flood Control message appears publically
      
  • 🇺🇸United States nmillin

    **Note** Hopefully this helps a future dev.

    I like that the message isn't being displayed anymore. Now an error message of "An error occurred while searching, try again later." will show for me. This is coming from SearchApiSolrBackend.php (for me at least).

    This message can be overridden by something like:

    /**
     * Implements hook_preprocess_HOOK().
     */
    function mymodule_preprocess_status_messages(&$variables) {
      // Normal error message from SearchApiSolrBackend.php. Could be better.
      if (isset($variables['message_list']['error'])) {
        $status_messages = $variables['message_list']['error'];
        foreach ($status_messages as $delta => $message) {
          if (strpos((string) $message, 'An error occurred while searching, try again later.') !== FALSE) {
            $variables['message_list']['error'][$delta] = "Search had a bad time. Call the help desk for assistance.";
          }
        }
      }
    }
    
  • Automatically closed - issue fixed for 2 weeks with no activity.

Production build 0.71.5 2024