CactuseSecurity / firewall-orchestrator

Network Security data repository for automation, reporting and compliance of firewall rules
https://fwo.cactus.de
Apache License 2.0
33 stars 11 forks source link

UI Reporting RSB - UI crashes on stest machine v8.4.1 #2589

Open tpurschke opened 1 month ago

tpurschke commented 1 month ago
SolidProgramming commented 1 month ago
  1. Where do i get test data for this?
  2. What does RSB mean? - I don't know this abbrevation yet @tpurschke
tpurschke commented 1 month ago
  1. Where do i get test data for this?

you may start with a 500 rules rulebase as referenced on our private repo: https://github.com/CactuseSecurity/FWO_private/blob/fba4f11f2b4401f075b69cdea73fd2d2cabad756/dev-infos/sample-data/cp500rules.md

  1. What does RSB mean? - I don't know this abbrevation yet

simply Right Side Bar :-) - sorry for the confusion

SolidProgramming commented 1 month ago

I dug a bit and this maybe could have to do with the new UI message "system" but not sure. If there are Ui messages while other UI stuff happens the message list could cause issues and if the task for the messages/loading of the data crash the stack trace maybe can't follow where the error happened and thus can't display any info for it. i will further investigate this tomorrow. @tpurschke

SolidProgramming commented 1 month ago

@tpurschke imported the 500 rules and this branch https://github.com/SolidProgramming/firewall-orchestrator/tree/2589-ui-reporing-rsb-ui-crashes-on-stest-machine-v841 doesn't break

tpurschke commented 3 weeks ago

@SolidProgramming can you please have a look at this? Questions seems to be - how do we get duplicate IDs here?

blazor.server.js:1 
 [2024-10-26T10:37:58.948Z] Error: System.InvalidOperationException: More than one sibling of element 'tr' has the same key value, 'FWO.Api.Data.NetworkObject'. Key values must be unique.
   at Microsoft.AspNetCore.Components.RenderTree.RenderTreeDiffBuilder.ThrowExceptionForDuplicateKey(Object key, RenderTreeFrame& frame)
   at Microsoft.AspNetCore.Components.RenderTree.RenderTreeDiffBuilder.BuildKeyToInfoLookup(DiffContext diffContext, Int32 oldStartIndex, Int32 oldEndIndexExcl, Int32 newStartIndex, Int32 newEndIndexExcl)
   at Microsoft.AspNetCore.Components.RenderTree.RenderTreeDiffBuilder.AppendDiffEntriesForRange(DiffContext& diffContext, Int32 oldStartIndex, Int32 oldEndIndexExcl, Int32 newStartIndex, Int32 newEndIndexExcl)
   at Microsoft.AspNetCore.Components.RenderTree.RenderTreeDiffBuilder.AppendDiffEntriesForFramesWithSameSequence(DiffContext& diffContext, Int32 oldFrameIndex, Int32 newFrameIndex)
   at Microsoft.AspNetCore.Components.RenderTree.RenderTreeDiffBuilder.AppendDiffEntriesForRange(DiffContext& diffContext, Int32 oldStartIndex, Int32 oldEndIndexExcl, Int32 newStartIndex, Int32 newEndIndexExcl)
   at Microsoft.AspNetCore.Components.RenderTree.RenderTreeDiffBuilder.AppendDiffEntriesForFramesWithSameSequence(DiffContext& diffContext, Int32 oldFrameIndex, Int32 newFrameIndex)
   at Microsoft.AspNetCore.Components.RenderTree.RenderTreeDiffBuilder.AppendDiffEntriesForRange(DiffContext& diffContext, Int32 oldStartIndex, Int32 oldEndIndexExcl, Int32 newStartIndex, Int32 newEndIndexExcl)
   at Microsoft.AspNetCore.Components.RenderTree.RenderTreeDiffBuilder.AppendDiffEntriesForFramesWithSameSequence(DiffContext& diffContext, Int32 oldFrameIndex, Int32 newFrameIndex)
   at Microsoft.AspNetCore.Components.RenderTree.RenderTreeDiffBuilder.AppendDiffEntriesForRange(DiffContext& diffContext, Int32 oldStartIndex, Int32 oldEndIndexExcl, Int32 newStartIndex, Int32 newEndIndexExcl)
   at Microsoft.AspNetCore.Components.Rendering.ComponentState.RenderIntoBatch(RenderBatchBuilder batchBuilder, RenderFragment renderFragment, Exception& renderFragmentException)
   at Microsoft.AspNetCore.Components.RenderTree.Renderer.ProcessRenderQueue()
log @   blazor.server.js:1
tpurschke commented 3 weeks ago

@Y4nnikH normalized test data which reproduces the issue can be found here: https://xfer.cactus.de/index.php/f/695816. pls. contact me if you have issues importing it.