Skip to content

Commit 24d7bc3

Browse files
authored
Merge pull request rmosolgo#4319 from rmosolgo/name-encoding
Introspection performance: encode names in UTF-8
2 parents 4712a3d + ec4054a commit 24d7bc3

2 files changed

Lines changed: 2 additions & 1 deletion

File tree

lib/graphql/schema/field.rb

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -229,6 +229,7 @@ def initialize(type: nil, name: nil, owner: nil, null: nil, description: :not_gi
229229
end
230230
@original_name = name
231231
name_s = -name.to_s
232+
232233
@underscored_name = -Member::BuildType.underscore(name_s)
233234
@name = -(camelize ? Member::BuildType.camelize(name_s) : name_s)
234235
if description != :not_given

lib/graphql/types/string.rb

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,7 @@ class String < GraphQL::Schema::Scalar
77

88
def self.coerce_result(value, ctx)
99
str = value.to_s
10-
if str.encoding == Encoding::UTF_8
10+
if str.ascii_only? || str.encoding == Encoding::UTF_8
1111
str
1212
elsif str.frozen?
1313
str.encode(Encoding::UTF_8)

0 commit comments

Comments
 (0)