Skip to content

fix: validate custom property name charset #27808

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.

Sign up for GitHub

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

Open
sonika-shah wants to merge 26 commits into
base: main
Choose a base branch
from

fix(it): request customProperties field on read-back in PATCH IT

d46af48
Select commit
Loading
Failed to load commit list.
Open

fix: validate custom property name charset #27808

fix(it): request customProperties field on read-back in PATCH IT
d46af48
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ui] SonarCloud Code Analysis succeeded May 5, 2026 in 5m 8s

Quality Gate passed

Issues
1062 New issues
0 Accepted issues

Measures
17 Security Hotspots
0.0% Coverage on New Code
3.8% Duplication on New Code

See analysis details on SonarQube Cloud

Annotations

Check failure on line 1342 in src/components/OntologyExplorer/hooks/useOntologyExplorer.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this code to not nest functions more than 4 levels deep. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOIznJnsbue2goZkv&open=AZ3YOIznJnsbue2goZkv&pullRequest=27808

Check failure on line 245 in src/utils/DataQuality/DataQualityUtils.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 24 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOKtaJnsbue2goZnW&open=AZ3YOKtaJnsbue2goZnW&pullRequest=27808

Check failure on line 1344 in src/components/OntologyExplorer/hooks/useOntologyExplorer.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this code to not nest functions more than 4 levels deep. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOIznJnsbue2goZkw&open=AZ3YOIznJnsbue2goZkw&pullRequest=27808

Check warning on line 52 in src/components/common/InlineAlert/InlineAlert.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

'subDescription' may use Object's default stringification format ('[object Object]') when stringified. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YODx3Jnsbue2goZdz&open=AZ3YODx3Jnsbue2goZdz&pullRequest=27808

Check warning on line 169 in src/components/MyData/Widgets/CuratedAssetsWidget/AdvancedAssetsFilterField/AdvancedAssetsFilterField.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Handle this exception or don't catch it at all. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOFI1Jnsbue2goZfY&open=AZ3YOFI1Jnsbue2goZfY&pullRequest=27808

Check warning on line 1017 in src/components/DataContract/ODCSImportModal/ODCSImportModal.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Do not use Array index in keys 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOITgJnsbue2goZju&open=AZ3YOITgJnsbue2goZju&pullRequest=27808

Check warning on line 88 in src/components/common/FocusTrap/FocusTrapWithContainer.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Avoid non-native interactive elements. If using native HTML is not possible, add an appropriate role and support for tabbing, mouse, keyboard, and touch inputs to an interactive content element. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOEKoJnsbue2goZeR&open=AZ3YOEKoJnsbue2goZeR&pullRequest=27808

Check warning on line 177 in src/components/OntologyExplorer/utils/hierarchyGraphBuilder.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOIkQJnsbue2goZkD&open=AZ3YOIkQJnsbue2goZkD&pullRequest=27808

Check warning on line 262 in src/components/Glossary/GlossaryHeader/GlossaryHeader.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOG4ZJnsbue2goZh8&open=AZ3YOG4ZJnsbue2goZh8&pullRequest=27808

Check failure on line 1351 in src/components/OntologyExplorer/hooks/useOntologyExplorer.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this code to not nest functions more than 4 levels deep. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOIznJnsbue2goZkx&open=AZ3YOIznJnsbue2goZkx&pullRequest=27808

Check warning on line 42 in src/components/BlockEditor/Extensions/File/AttachmentComponents/FileAttachment.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Visible, non-interactive elements with click handlers must have at least one keyboard listener. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YN4qJJnsbue2goZbC&open=AZ3YN4qJJnsbue2goZbC&pullRequest=27808

Check failure on line 826 in src/components/OntologyExplorer/hooks/useOntologyGraph.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 38 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOIu9Jnsbue2goZkg&open=AZ3YOIu9Jnsbue2goZkg&pullRequest=27808

Check warning on line 406 in src/components/common/UserTeamSelectableList/UserTeamSelectableList.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Unexpected negated condition. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOD-2Jnsbue2goZeH&open=AZ3YOD-2Jnsbue2goZeH&pullRequest=27808

Check warning on line 99 in src/components/DataQuality/SummaryPannel/SummaryPieChartCard/SummaryPieChartCard.component.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Do not use Array index in keys 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOE9WJnsbue2goZfJ&open=AZ3YOE9WJnsbue2goZfJ&pullRequest=27808

Check warning on line 510 in src/utils/CanvasUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `Number.parseFloat` over `parseFloat`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOK_tJnsbue2goZn0&open=AZ3YOK_tJnsbue2goZn0&pullRequest=27808

Check warning on line 40 in src/hooks/useClipBoard.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `childNode.remove()` over `parentNode.removeChild(childNode)`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOMieJnsbue2goZrG&open=AZ3YOMieJnsbue2goZrG&pullRequest=27808

Check failure on line 445 in src/components/DataContract/ContractSecurityFormTab/ContractSecurityFormTab.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this code to not nest functions more than 4 levels deep. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOIOhJnsbue2goZjj&open=AZ3YOIOhJnsbue2goZjj&pullRequest=27808

Check warning on line 309 in src/hooks/useGridEditController.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `Number.parseInt` over `parseInt`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOMpfJnsbue2goZrX&open=AZ3YOMpfJnsbue2goZrX&pullRequest=27808

Check warning on line 87 in src/components/DataInsight/DataInsightSummary.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Handle this exception or don't catch it at all. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOH_oJnsbue2goZjO&open=AZ3YOH_oJnsbue2goZjO&pullRequest=27808

Check warning on line 120 in src/components/MyData/Widgets/KPIWidget/KPILegend/KPILegend.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Extract this nested ternary operation into an independent statement. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOFTuJnsbue2goZfk&open=AZ3YOFTuJnsbue2goZfk&pullRequest=27808

Check failure on line 603 in src/components/OntologyExplorer/hooks/useOntologyExplorer.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Refactor this function to reduce its Cognitive Complexity from 16 to the 15 allowed. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOIznJnsbue2goZks&open=AZ3YOIznJnsbue2goZks&pullRequest=27808

Check warning on line 508 in src/utils/CanvasUtils.ts

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `Number.parseFloat` over `parseFloat`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOK_tJnsbue2goZny&open=AZ3YOK_tJnsbue2goZny&pullRequest=27808

Check warning on line 69 in src/components/common/FormBuilderV1/FormBuilderV1.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer using an optional chain expression instead, as it's more concise and easier to read. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YODfTJnsbue2goZdG&open=AZ3YODfTJnsbue2goZdG&pullRequest=27808

Check warning on line 162 in src/components/WorkflowDefinitions/WorkflowBuilder/forms/UserApprovalForm.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `Number.isNaN` over `isNaN`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOF15Jnsbue2goZgW&open=AZ3YOF15Jnsbue2goZgW&pullRequest=27808

Check failure on line 315 in src/utils/SchedularUtils.tsx

See this annotation in the file changed.

@sonarqubecloud sonarqubecloud / [open-metadata-ui] SonarCloud Code Analysis

Prefer `throw error` over `return Promise.reject(error)`. 

See more on https://sonarcloud.io/project/issues?id=open-metadata-ui&issues=AZ3YOL9eJnsbue2goZp9&open=AZ3YOL9eJnsbue2goZp9&pullRequest=27808
View more details on SonarQubeCloud