if a child span with no tag is reported, UI crashes when trying to expand it

[primalmotion]

reporting a span with no tag causes the UI to crash when trying to access the span detail in the trace view.

SpanDetail.js:77 Uncaught TypeError: Cannot read property 'map' of undefined
    at a (SpanDetail.js:77)
    at f._constructComponentWithoutOwner (ReactCompositeComponent.js:309)
    at f._constructComponent (ReactCompositeComponent.js:285)
    at f.mountComponent (ReactCompositeComponent.js:188)
    at Object.mountComponent (ReactReconciler.js:46)
    at f.performInitialMount (ReactCompositeComponent.js:371)
    at f.mountComponent (ReactCompositeComponent.js:258)
    at Object.mountComponent (ReactReconciler.js:46)
    at h.mountChildren (ReactMultiChild.js:238)
    at h._createInitialChildren (ReactDOMComponent.js:697)

Steps to reproduce the issue:

1. create a child span with no tag
2. access the detail view in the ui
3. click on the child span
4. the crash occurs.

Describe the results you received:
the crash

Describe the results you expected:
no crash

Additional information you deem important (e.g. issue happens only occasionally):

Additional environment details (Browser, etc.):
Chrome, but I don't think it matters

[yurishkuro]

are you using the latest image, like 0.6.0? This bug has been fixed quite some time ago (at least sounds like that bug)

[primalmotion]

I’m using jaeger-query 0.7.0

[tiffon]

I haven't been able to reproduce this. But, it should be fixed (I put code in place to handle it, anyway). After we test the latest Jaeger UI internally, we should cut a new release, which will have the fix. I'll post back, here, when we cut the new docker image.

[tiffon]

This is resolved. @yurishkuro When are we planning to cut a new all-in-one?

[yurishkuro]

we can, just need a changelog for accumulated UI changes

[tiffon]

Got it. 👍

…

On Oct 20, 2017 6:12 PM, "Yuri Shkuro" ***@***.***> wrote: we can, just need a changelog for accumulated UI changes — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub <https://github.com/uber/jaeger-ui/issues/70#issuecomment-338334249>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACMpUbFiZOb-Wjrf7yl9TF2FmAFpfEWMks5suRrBgaJpZM4PJVRd> .