Howdy! I'm using this lib in conjunction w/ Fauna functions to make

Hey <a class="user-mention notranslate" data-hovercard-type="user" data-hovercard-url=

Errors lose nested error descriptions about faunadb-go HOT 2 OPEN

fauna commented on May 26, 2024

Errors lose nested error descriptions

from faunadb-go.

Comments (2)

vadimLF commented on May 26, 2024

Hey @wlockiv. Nice catch.
We prepare PR #135 which fix this but I put this PR on hold since we want some consistency for errors output for all drivers (go, jvm, etc.).

from faunadb-go.

ssttevee commented on May 26, 2024

As an extension to this, the Cause field should just be fully recursive.

I have UDFs with many levels of nesting, and this is what I've seen and have been unable to catch:

{
  errors: [
    {
      position: [],
      code: 'call error',
      description: 'Calling the function resulted in an error.',
      cause: [
        {
          position: [
            'expr',
            'in',
            'from',
            1
          ],
          code: 'call error',
          description: 'Calling the function resulted in an error.',
          cause: [
            {
              position: [
                'expr',
                'in',
                'from',
                2
              ],
              code: 'call error',
              description: 'Calling the function resulted in an error.',
              cause: [
                {
                  position: [
                    'expr',
                    'in',
                    'from',
                    2,
                    'lambda'
                  ],
                  code: 'invalid argument',
                  description: 'Lambda expects an array with 5 elements. Array contains 4.'
                }
              ]
            }
          ]
        }
      ]
    }
  ]
}

Since ValidationFailure is just the same as QueryError minus Cause, I think it would be best to change the type of Cause to []QueryError instead of []ValidationFailure.

from faunadb-go.

Recommend Projects