open-formulieren / open-forms

Smart and dynamic forms
https://open-forms.readthedocs.io
Other
37 stars 26 forks source link

Daily digest reporting prefill failures for forms that work with more then 1 auth "flow" #4825

Open LaurensBurger opened 6 days ago

LaurensBurger commented 6 days ago

Product versie / Product version

2.7.8 - 2.8.1

Customer reference

SED 102

Omschrijf het probleem / Describe the bug

Most environments use a common NAW-stap that allows for multiple authentication: DigiD + eHerkenning and anonymous.

These steps usually consist of 2 or 3 field sets and depending on which form of authentication is used, the right field set will be shown with pre-filled values.

In a situation where a users logs into a form with DigiD a request is made to fetch user data from the BRP, and not to the KvK to fetch the business data.

However in the Logs we can see something is going on, but wrongly log a request has been made and the response was empty. These log items also get reported in the daily digest as failures.

Example with eHerkenning login: image

Example with DigiD login: image

Both submissions show 1 failed and 1 successful pre-fill attempt. But in both causes the failed attempt seems to be "false" (we don't see any outgoing requests, no sentry errors)

As these steps/forms are used on almost all environments, the extra "failures" on the daily digest cause confusion and hinders the visibility of actual issues.

sergei-maertens commented 6 days ago

we don't see any outgoing requests, no sentry errors

You probably won't ever see sentry errors for this because any errors are suppressed because of a "continue at all costs" philosophy in prefill :grimacing:

joeribekker commented 3 days ago

Refinement: This might be caused by returning an empty list because there is no kvk/bsn which is inadvertently marked as "problem" log entry. There is also overlap between this feature and configuration problem detection. This is some technical debt that needs to be addressed.

Let's first investigate if we can fix it quickly. Please report back if this is not the case.