diff options
author | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2019-04-24 14:39:07 +0300 |
---|---|---|
committer | Dmitriy Zaporozhets <dmitriy.zaporozhets@gmail.com> | 2019-04-24 14:39:07 +0300 |
commit | 8e418477879024ec0074dafe45289ab463de93e5 (patch) | |
tree | b09ab92a6a71c221205fc12f010b4521db7aa519 /lib | |
parent | d056d9d4b78230d16bdda2557434f50bc929a657 (diff) | |
parent | eff42d59bd1e0abdab673f5fca73112826af49cd (diff) |
Merge branch '60800-properly-authorize-our-own-graphql-scalar-types' into 'master'
Properly authorize our own GraphQL scalar types
Closes #60800
See merge request gitlab-org/gitlab-ce!27563
Diffstat (limited to 'lib')
-rw-r--r-- | lib/gitlab/graphql/authorize/authorize_field_service.rb | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/gitlab/graphql/authorize/authorize_field_service.rb b/lib/gitlab/graphql/authorize/authorize_field_service.rb index 03d6aabb0e3..619ce100421 100644 --- a/lib/gitlab/graphql/authorize/authorize_field_service.rb +++ b/lib/gitlab/graphql/authorize/authorize_field_service.rb @@ -48,7 +48,7 @@ module Gitlab end def authorize_against(parent_typed_object, resolved_type) - if built_in_type? + if scalar_type? # The field is a built-in/scalar type, or a list of scalars # authorize using the parent's object parent_typed_object.object @@ -108,8 +108,8 @@ module Gitlab type.unwrap end - def built_in_type? - GraphQL::Schema::BUILT_IN_TYPES.has_value?(node_type_for_basic_connection(@field.type)) + def scalar_type? + node_type_for_basic_connection(@field.type).kind.scalar? end end end |