Skip to content

refactor(keycloak): migrate Keycloak plugin to NestJS #1973

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

Closed
shikanime wants to merge 3 commits into from

refactor(keycloak): migrate Keycloak plugin to NestJS

a30f81e
Select commit
Loading
Failed to load commit list.
Closed

refactor(keycloak): migrate Keycloak plugin to NestJS #1973

refactor(keycloak): migrate Keycloak plugin to NestJS
a30f81e
Select commit
Loading
Failed to load commit list.
cloud-pi-native-sonarqube / SonarQube Code Analysis succeeded Mar 12, 2026 in 49s

Quality Gate success

Passed Quality Gate passed

Issues

Measures

Project ID: cloud-pi-native_console_AYsa46O31ebDtzs-pYyZ

View in SonarQube

Annotations

Check notice on line 131 in apps/server-nestjs/src/modules/keycloak/keycloak.service.ts

See this annotation in the file changed.

@cloud-pi-native-sonarqube cloud-pi-native-sonarqube / SonarQube Code Analysis

apps/server-nestjs/src/modules/keycloak/keycloak.service.ts#L131 

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

Check notice on line 55 in apps/server-nestjs/src/modules/keycloak/keycloak.service.ts

See this annotation in the file changed.

@cloud-pi-native-sonarqube cloud-pi-native-sonarqube / SonarQube Code Analysis

apps/server-nestjs/src/modules/keycloak/keycloak.service.ts#L55 

Unexpected negated condition.

Check failure on line 51 in apps/server-nestjs/src/modules/keycloak/keycloak.service.ts

See this annotation in the file changed.

@cloud-pi-native-sonarqube cloud-pi-native-sonarqube / SonarQube Code Analysis

apps/server-nestjs/src/modules/keycloak/keycloak.service.ts#L51 

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

Check notice on line 127 in apps/server-nestjs/src/modules/keycloak/keycloak.service.ts

See this annotation in the file changed.

@cloud-pi-native-sonarqube cloud-pi-native-sonarqube / SonarQube Code Analysis

apps/server-nestjs/src/modules/keycloak/keycloak.service.ts#L127 

Expected a `for-of` loop instead of a `for` loop with this simple iteration.

Check notice on line 34 in apps/server-nestjs/src/modules/keycloak/keycloak.service.ts

See this annotation in the file changed.

@cloud-pi-native-sonarqube cloud-pi-native-sonarqube / SonarQube Code Analysis

apps/server-nestjs/src/modules/keycloak/keycloak.service.ts#L34 

Complete the task associated to this "TODO" comment.

Check notice on line 129 in apps/server-nestjs/src/modules/keycloak/keycloak.service.ts

See this annotation in the file changed.

@cloud-pi-native-sonarqube cloud-pi-native-sonarqube / SonarQube Code Analysis

apps/server-nestjs/src/modules/keycloak/keycloak.service.ts#L129 

Unexpected negated condition.
View more details on cloud-pi-native-sonarqube