Shield rules with cache: "strict" don't work on types.

Bug report

• [x ] I have checked other issues to make sure this is not a duplicate.

Describe the bug

When using shield rule on type with cache: "strict" authorization result is always “Not Authorized”

To Reproduce

Steps to reproduce the behavior, please provide code snippets or a repository:

(Delete the filler code and replace it with your own)

1. This is my GraphQL Schema.

type Query {
  clinics: [Clinic]!
}

type Clinic {
  id: ID!
  name: String!
}

1. This is the invoked query

clinics {
  id
  name
}

1. I use these permissions

const permissions = shield({
  Query: {
    clinics: allow,
  },
  Clinic: rule({ cache: 'strict' })(async (parent, args, ctx, info) => {
    return true;
  }),,
})

1. All results are not authorized.

Expected behavior

Rule should be called once per “Type” and validate access to it.

Actual behaviour

Rule always return “Not authorized”

Additional

If this is wrong usage of rules then it should be somehow thrown as an error

[todda00]

I can’t seem to recreate this issue anymore either - I have set all my rules back from no_cache to strict which were strict previously. As we test the app I will let you know if anything pops up, but I can’t recreate it inside our app, or in the sandbox you provided.

[todda00]

I have run into this issue, but specifically only when a parent is null and cache === “strict”. The rule does not even run at all. When debug=true, the following error is output:

Cannot read property '3' of undefined

stacktrace:

TypeError: Cannot read property '3' of undefined,
    at Object.get [as generationTime] ([redacted]/node_modules/bson/lib/bson/objectid.js:373:19),
    at [redacted]/node_modules/object-hash/index.js:246:33,
    at Array.forEach (<anonymous>),
    at Object._object ([redacted]/node_modules/object-hash/index.js:242:21),
    at Object.dispatch ([redacted]/node_modules/object-hash/index.js:185:30),
    at [redacted]/node_modules/object-hash/index.js:246:18,
    at Array.forEach (<anonymous>),
    at Object._object ([redacted]/node_modules/object-hash/index.js:242:21),
    at Object.dispatch ([redacted]/node_modules/object-hash/index.js:185:30),
    at [redacted]/node_modules/object-hash/index.js:246:18,
    at Array.forEach (<anonymous>),
    at Object._object ([redacted]/node_modules/object-hash/index.js:242:21),
    at Object.dispatch ([redacted]/node_modules/object-hash/index.js:185:30),
    at [redacted]/node_modules/object-hash/index.js:246:18,
    at Array.forEach (<anonymous>),
    at Object._object ([redacted]/node_modules/object-hash/index.js:242:21),
    at Object.dispatch ([redacted]/node_modules/object-hash/index.js:185:30),
    at hash ([redacted]/node_modules/object-hash/index.js:128:10),
    at Object.objectHash [as hashFunction] ([redacted]/node_modules/object-hash/index.js:33:10),
    at Rule.executeRule ([redacted]/node_modules/graphql-shield/src/rules.ts:162:33),
    at Rule.resolve ([redacted]/node_modules/graphql-shield/src/rules.ts:58:30),
    at middleware ([redacted]/node_modules/graphql-shield/src/generator.ts:67:30),
    at [redacted]/node_modules/graphql-middleware/src/applicator.ts:32:5,
    at field.resolve ([redacted]/node_modules/apollo-server-core/node_modules/graphql-extensions/src/index.ts:274:18),
    at resolveFieldValueOrError ([redacted]/node_modules/graphql/execution/execute.js:503:18),
    at resolveField ([redacted]/node_modules/graphql/execution/execute.js:470:16),
    at executeFields ([redacted]/node_modules/graphql/execution/execute.js:311:18),
    at collectAndExecuteSubfields ([redacted]/node_modules/graphql/execution/execute.js:747:10),
    at completeObjectValue ([redacted]/node_modules/graphql/execution/execute.js:737:10),
    at completeValue ([redacted]/node_modules/graphql/execution/execute.js:626:12),
    at [redacted]/node_modules/graphql/execution/execute.js:528:16,
    at process._tickCallback (internal/process/next_tick.js:68:7)