microsoft / coe-starter-kit

Other
754 stars 225 forks source link

[CoE Starter Kit - BUG] Invalid Character in admin_displayname - CoE BYODL Apps Dataflow #7943

Open nasilvae opened 7 months ago

nasilvae commented 7 months ago

Does this bug already exist in our backlog?

Describe the issue

CoE BYODL Apps Dataflow is erroring out due to an invalid character in the admin_displayname.

image image

Expected Behavior

Dataflow should validate normally and work.

What solution are you experiencing the issue with?

Core

What solution version are you using?

4.24

What app or flow are you having the issue with?

CoE BYODL Apps Dataflow

What method are you using to get inventory and telemetry?

Data Export

Steps To Reproduce

No response

Anything else?

No response

AB#2673

nasilvae commented 7 months ago

We added a replace " \ " with "" step in the admin_makers table inside the apps dataflow. Seems to be working but verifying right now.

Jenefer-Monroe commented 7 months ago

Please do let me know the character that you believe was causing issues so I can test locally. I have some special character test users and dont repro so would like to be able to see the issue locally before we make a change here.

nasilvae commented 7 months ago

Please do let me know the character that you believe was causing issues so I can test locally. I have some special character test users and dont repro so would like to be able to see the issue locally before we make a change here.

I tried filtering the makers table in the dataflow and deleted the refresh history but I still get the same error after we reran the dataflows after clearing the dataflowrefreshhistory.

the value is "\b"

image

Jenefer-Monroe commented 7 months ago

Thanks we'll create a test user and explore to see if we can fix this up

Jenefer-Monroe commented 7 months ago

pushing to the FC for data export updates