Skip to content

Improvements to SSO #27446

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
harshach wants to merge 5 commits into
base: main
Choose a base branch
from

Merge branch 'main' into sso_improvements

63d9383
Select commit
Loading
Failed to load commit list.
Open

Improvements to SSO #27446

Merge branch 'main' into sso_improvements
63d9383
Select commit
Loading
Failed to load commit list.
SonarQubeCloud / [open-metadata-ui] SonarCloud Code Analysis succeeded Apr 22, 2026 in 3m 49s

Quality Gate passed

Issues
996 New issues
0 Accepted issues

Measures
15 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 65 in src/components/BlockEditor/Extensions/File/FileNode.ts

See this annotation in the file changed.

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

Prefer `.dataset` over `getAttribute(…)`. 

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

Check warning on line 285 in src/components/Entity/EntityLineage/CustomNode.utils.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=AZ2ZETn60EzrWDwSuXJb&open=AZ2ZETn60EzrWDwSuXJb&pullRequest=27446

Check warning on line 16 in src/utils/EnvironmentUtils.ts

See this annotation in the file changed.

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

Compare with `undefined` directly instead of using `typeof`. 

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

Check warning on line 417 in src/components/ActivityFeed/ActivityFeedCardNew/ActivityFeedcardNew.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=AZ2ZETQ10EzrWDwSuXID&open=AZ2ZETQ10EzrWDwSuXID&pullRequest=27446

Check warning on line 64 in src/components/common/NavigationBlocker/NavigationBlocker.tsx

See this annotation in the file changed.

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

'returnValue' is deprecated. 

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

Check warning on line 702 in src/components/DataQuality/AddDataQualityTest/components/TestCaseFormV1.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=AZ2ZETAq0EzrWDwSuXHC&open=AZ2ZETAq0EzrWDwSuXHC&pullRequest=27446

Check failure on line 54 in src/utils/SwMessenger.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=AZ2ZEVAy0EzrWDwSuXQH&open=AZ2ZEVAy0EzrWDwSuXQH&pullRequest=27446

Check warning on line 2033 in src/constants/mockTourData.constants.ts

See this annotation in the file changed.

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

Don't use a zero fraction in the number. 

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

Check warning on line 180 in src/components/OntologyExplorer/utils/graphStyles.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=AZ2ZEUBW0EzrWDwSuXLR&open=AZ2ZEUBW0EzrWDwSuXLR&pullRequest=27446

Check warning on line 740 in src/utils/EntityLineageUtils.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=AZ2ZEVHq0EzrWDwSuXRJ&open=AZ2ZEVHq0EzrWDwSuXRJ&pullRequest=27446

Check warning on line 118 in src/components/common/Table/TableV2Utils.ts

See this annotation in the file changed.

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

This assertion is unnecessary since it does not change the type of the expression. 

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

Check warning on line 149 in src/utils/TagsUtils.tsx

See this annotation in the file changed.

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

Prefer `Array#flat()` over `Array#reduce()` to flatten an array. 

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

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

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=AZ2ZEVaZ0EzrWDwSuXUG&open=AZ2ZEVaZ0EzrWDwSuXUG&pullRequest=27446

Check warning on line 156 in src/components/common/UserTeamSelect/UserTeamSelect.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=AZ2ZES4u0EzrWDwSuXGm&open=AZ2ZES4u0EzrWDwSuXGm&pullRequest=27446

Check warning on line 93 in src/components/DataInsight/KPILatestResultsV1.tsx

See this annotation in the file changed.

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

Prefer `Math.max()` to simplify ternary expressions. 

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

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

See this annotation in the file changed.

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

Prefer `globalThis` over `window`. 

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

Check warning on line 130 in src/components/DomainListing/components/DomainTreeView.tsx

See this annotation in the file changed.

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

Complete the task associated to this "TODO" comment. 

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

Check warning on line 102 in src/components/WorkflowDefinitions/WorkflowBuilder/NodeConfigSidebar.tsx

See this annotation in the file changed.

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

Prefer using nullish coalescing operator (`??`) instead of a ternary expression, as it is simpler to read. 

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

Check failure on line 355 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=AZ2ZET7f0EzrWDwSuXKw&open=AZ2ZET7f0EzrWDwSuXKw&pullRequest=27446

Check warning on line 78 in src/utils/EdgeStyleUtils.ts

See this annotation in the file changed.

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

Function 'computeEdgeStyle' has too many parameters (10). Maximum allowed is 7. 

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

Check failure on line 755 in src/components/DataQuality/AddDataQualityTest/components/TestCaseFormV1.tsx

See this annotation in the file changed.

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

Prefer `return value` over `return Promise.resolve(value)`. 

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

Check warning on line 145 in src/components/GlobalSearchBar/GlobalSearchBar.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=AZ2ZEUOg0EzrWDwSuXMg&open=AZ2ZEUOg0EzrWDwSuXMg&pullRequest=27446

Check warning on line 156 in src/pages/AppInstall/AppInstall.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=AZ2ZEUWW0EzrWDwSuXM4&open=AZ2ZEUWW0EzrWDwSuXM4&pullRequest=27446

Check warning on line 385 in src/components/Explore/ExploreTree/ExploreTree.tsx

See this annotation in the file changed.

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

Move this component definition out of the parent component and pass data as props. 

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

Check warning on line 488 in src/components/ExploreV1/ExploreV1.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=AZ2ZEUHs0EzrWDwSuXMG&open=AZ2ZEUHs0EzrWDwSuXMG&pullRequest=27446
View more details on SonarQubeCloud