Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Error thrown when corrupted native dashboard filter metadata referencing obsolete filter ID #24729

Closed
3 tasks done
john-bodley opened this issue Jul 18, 2023 · 1 comment
Closed
3 tasks done
Assignees

Comments

@john-bodley
Copy link
Member

The dashboard native filters throws an unexpected error due to corrupted dashboard metadata.

How to reproduce the bug

Currently we're uncertain how the metadata becomes corrupted, though it likely occurs when people delete a filter. The issue surfaces when a the cascadeParentIds array references an ID which is not associated with any filters, e.g.

{
    "native_filter_configuration": [
        {
            "id": "NATIVE_FILTER-4vk933fy2",
            "cascadeParentIds": [
                "NATIVE_FILTER-Fvk_h6yN"
             ],
             ...
        }
    ]  
}

Removing the hanging NATIVE_FILTER-Fvk_h6yN reference remedies the issue.

Expected results

The cascadeParentIds array should only contain the IDs of currently defined filters.

Actual results

An error is thrown.

Screenshots

error

Environment

(please complete the following information):

  • browser type and version: Chrome 114.0.5735.198
  • superset version: master
  • python version: 3.9
  • node.js version: 16
  • any feature flags active:

Checklist

Make sure to follow these steps before submitting your issue - thank you!

  • I have checked the superset logs for python stacktraces and included it here as text if there are any.
  • I have reproduced the issue with at least the latest released version of superset.
  • I have checked the issue tracker for the same issue and I haven't found one similar.

Additional context

Add any other context about the problem here.

@michael-s-molina
Copy link
Member

Closed by #24749

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants