SendGrid - NoMethodError (undefined method `[]' for nil:NilClass)

cultiv · February 23, 2025, 8:48am

Did you ever figure this one out? I have the same error, on the same line of code:

app/controllers/webhooks_controller.rb

main


      
              "Received a Sendgrid webhook, but no verification key has been configured. This is unsafe behaviour and will be disallowed in the future.",
            )
          end
          
          events = params["_json"] || [params]
          events.each do |event|
            message_id = Email::MessageIdService.message_id_clean((event["smtp-id"] || ""))
            to_address = event["email"]
            error_code = event["status"]
            if event["event"] == "bounce"
              if error_code[Email::SMTP_STATUS_TRANSIENT_FAILURE]
                process_bounce(message_id, to_address, SiteSetting.soft_bounce_score, error_code)
              else
                process_bounce(message_id, to_address, SiteSetting.hard_bounce_score, error_code)
              end
            elsif event["event"] == "dropped"
              process_bounce(message_id, to_address, SiteSetting.hard_bounce_score, error_code)
            end
          end
          
          success

I am not familiar with Ruby but it looks like it can parse the json just fiine up until trrying to parse the error code, Email::SMTP_STATUS_TRANSIENT_FAILURE points to:

github.com/discourse/discourse

lib/email.rb

main


      
          # frozen_string_literal: true
          
          require "mail"
          
          module Email
            # See https://www.iana.org/assignments/smtp-enhanced-status-codes/smtp-enhanced-status-codes.xhtml#smtp-enhanced-status-codes-1
            SMTP_STATUS_SUCCESS = "2."
            SMTP_STATUS_TRANSIENT_FAILURE = "4."
            SMTP_STATUS_PERMANENT_FAILURE = "5."
          
            def self.is_valid?(email)
              return false unless String === email
              EmailAddressValidator.valid_value?(email)
            end
          
            def self.downcase(email)
              return email unless Email.is_valid?(email)
              email.downcase

I checked in Webhook.site what Sendgrid actually sends when I test the webhook and it looks like this for a bounce:

[
  {
    "email": "example@test.com",
    "timestamp": 1740136261,
    "smtp-id": "<14c5d75ce93.dfd.64b469@ismtpd-555>",
    "event": "bounce",
    "category": [
      "cat facts"
    ],
    "sg_event_id": "ovGQ2rRo8ytNezHPDq-7Ig==",
    "sg_message_id": "14c5d75ce93.dfd.64b469.filter0001.16648.5515E0B88.0",
    "reason": "500 unknown recipient",
    "status": "5.0.0"
  }
]

Seems like it should work!

Topic		Replies	Views
Sendgrid error: NoMethodError (undefined method `[]' for nil:NilClass) Support	4	1134	November 5, 2020
Email::Processor does not expect to be called with mail=nil Bug email	14	1060	May 20, 2023
Severe Email Issue since last update a couple days ago - 3.4.0.beta4-dev Support	25	262	February 16, 2025
Starting new topics by email - Job exception: execution expired Support	6	2129	January 1, 2019
Digest send failures since update (NoMethodError) Bug activity-summary	25	1474	November 4, 2020

SendGrid - NoMethodError (undefined method `[]' for nil:NilClass)

Related topics