From 96f51e593f2609579b8155d971bcbc5ab9e7cd4c Mon Sep 17 00:00:00 2001
From: Emily Strickland <github@emily.st>
Date: Fri, 11 Nov 2022 12:22:28 -0800
Subject: [PATCH] Guard against error extracting `body` from URL (#20428)
If `Nokogiri::HTML(value).at_xpath('//body')` fails to find the `body` element, it will return `nil`. We can guard against that with an early return. Avoids calling `children` on `Nilclass` in those cases.
---
app/models/account/field.rb | 1 +
1 file changed, 1 insertion(+)
diff --git a/app/models/account/field.rb b/app/models/account/field.rb
index e84a0eeb1c..ffc8dce80b 100644
--- a/app/models/account/field.rb
+++ b/app/models/account/field.rb
@@ -76,6 +76,7 @@ class Account::Field < ActiveModelSerializers::Model
def extract_url_from_html
doc = Nokogiri::HTML(value).at_xpath('//body')
+ return if doc.nil?
return if doc.children.size > 1
element = doc.children.first
--
GitLab