Skip to content
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.

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

MODELIX-1042 authorization for workspaces #1190

Merged
merged 20 commits into from
Dec 12, 2024

fix(authorization): allow only resource owners/admins to manage permi…

817f454
Select commit
Loading
Failed to load commit list.
Merged

MODELIX-1042 authorization for workspaces #1190

fix(authorization): allow only resource owners/admins to manage permi…
817f454
Select commit
Loading
Failed to load commit list.
GitHub Advanced Security / detekt succeeded Dec 12, 2024 in 8s

138 new alerts

New alerts in code changed by this pull request

  • 138 warnings

See annotations below for details.

View all branch alerts.

Annotations

Check warning on line 7 in authorization/src/main/kotlin/org/modelix/authorization/AccessTokenPrincipal.kt

See this annotation in the file changed.

Code scanning / detekt

The function getUserName is missing documentation. Warning

The function getUserName is missing documentation.

Check warning on line 143 in authorization/src/main/kotlin/org/modelix/authorization/AuthorizationConfig.kt

See this annotation in the file changed.

Code scanning / detekt

The property jwtUtil is missing documentation. Warning

The property jwtUtil is missing documentation.

Check warning on line 205 in authorization/src/main/kotlin/org/modelix/authorization/AuthorizationConfig.kt

See this annotation in the file changed.

Code scanning / detekt

Companion is missing required documentation. Warning

Companion is missing required documentation.

Check warning on line 227 in authorization/src/main/kotlin/org/modelix/authorization/AuthorizationConfig.kt

See this annotation in the file changed.

Code scanning / detekt

The function ensureMinSecretLength is missing documentation. Warning

The function ensureMinSecretLength is missing documentation.

Check warning on line 231 in authorization/src/main/kotlin/org/modelix/authorization/AuthorizationConfig.kt

See this annotation in the file changed.

Code scanning / detekt

This expression contains a magic number. Consider defining it to a well named constant. Warning

This expression contains a magic number. Consider defining it to a well named constant.

Check warning on line 238 in authorization/src/main/kotlin/org/modelix/authorization/AuthorizationConfig.kt

See this annotation in the file changed.

Code scanning / detekt

This expression contains a magic number. Consider defining it to a well named constant. Warning

This expression contains a magic number. Consider defining it to a well named constant.

Check warning on line 245 in authorization/src/main/kotlin/org/modelix/authorization/AuthorizationConfig.kt

See this annotation in the file changed.

Code scanning / detekt

This expression contains a magic number. Consider defining it to a well named constant. Warning

This expression contains a magic number. Consider defining it to a well named constant.

Check warning on line 176 in authorization/src/main/kotlin/org/modelix/authorization/AuthorizationPlugin.kt

See this annotation in the file changed.

Code scanning / detekt

The function getDeniedPermissions is missing documentation. Warning

The function getDeniedPermissions is missing documentation.

Check warning on line 182 in authorization/src/main/kotlin/org/modelix/authorization/AuthorizationPlugin.kt

See this annotation in the file changed.

Code scanning / detekt

The function hasPermission is missing documentation. Warning

The function hasPermission is missing documentation.

Check warning on line 197 in authorization/src/main/kotlin/org/modelix/authorization/AuthorizationPlugin.kt

See this annotation in the file changed.

Code scanning / detekt

This expression contains a magic number. Consider defining it to a well named constant. Warning

This expression contains a magic number. Consider defining it to a well named constant.

Check warning on line 230 in authorization/src/main/kotlin/org/modelix/authorization/AuthorizationPlugin.kt

See this annotation in the file changed.

Code scanning / detekt

DeniedPermissionRequest is missing required documentation. Warning

DeniedPermissionRequest is missing required documentation.

Check warning on line 231 in authorization/src/main/kotlin/org/modelix/authorization/AuthorizationPlugin.kt

See this annotation in the file changed.

Code scanning / detekt

The property permissionRef is missing documentation. Warning

The property permissionRef is missing documentation.

Check warning on line 232 in authorization/src/main/kotlin/org/modelix/authorization/AuthorizationPlugin.kt

See this annotation in the file changed.

Code scanning / detekt

The property userId is missing documentation. Warning

The property userId is missing documentation.

Check warning on line 233 in authorization/src/main/kotlin/org/modelix/authorization/AuthorizationPlugin.kt

See this annotation in the file changed.

Code scanning / detekt

The property jwtPayload is missing documentation. Warning

The property jwtPayload is missing documentation.

Check warning on line 235 in authorization/src/main/kotlin/org/modelix/authorization/AuthorizationPlugin.kt

See this annotation in the file changed.

Code scanning / detekt

The function jwtPayloadJson is missing documentation. Warning

The function jwtPayloadJson is missing documentation.

Check warning on line 5 in authorization/src/main/kotlin/org/modelix/authorization/IAccessControlDataProvider.kt

See this annotation in the file changed.

Code scanning / detekt

IAccessControlDataProvider is missing required documentation. Warning

IAccessControlDataProvider is missing required documentation.

Check warning on line 6 in authorization/src/main/kotlin/org/modelix/authorization/IAccessControlDataProvider.kt

See this annotation in the file changed.

Code scanning / detekt

The function getGrantedPermissionsForUser is missing documentation. Warning

The function getGrantedPermissionsForUser is missing documentation.

Check warning on line 7 in authorization/src/main/kotlin/org/modelix/authorization/IAccessControlDataProvider.kt

See this annotation in the file changed.

Code scanning / detekt

The function getGrantedPermissionsForRole is missing documentation. Warning

The function getGrantedPermissionsForRole is missing documentation.

Check warning on line 10 in authorization/src/main/kotlin/org/modelix/authorization/IAccessControlDataProvider.kt

See this annotation in the file changed.

Code scanning / detekt

EmptyAccessControlDataProvider is missing required documentation. Warning

EmptyAccessControlDataProvider is missing required documentation.

Check warning on line 3 in authorization/src/main/kotlin/org/modelix/authorization/KeycloakTokenConstants.kt

See this annotation in the file changed.

Code scanning / detekt

KeycloakTokenConstants is missing required documentation. Warning

KeycloakTokenConstants is missing required documentation.

Check warning on line 4 in authorization/src/main/kotlin/org/modelix/authorization/KeycloakTokenConstants.kt

See this annotation in the file changed.

Code scanning / detekt

The property EMAIL is missing documentation. Warning

The property EMAIL is missing documentation.

Check warning on line 4 in authorization/src/main/kotlin/org/modelix/authorization/KeycloakTokenConstants.kt

See this annotation in the file changed.

Code scanning / detekt

EMAIL can be a const val. Warning

EMAIL can be a const val.

Check warning on line 5 in authorization/src/main/kotlin/org/modelix/authorization/KeycloakTokenConstants.kt

See this annotation in the file changed.

Code scanning / detekt

The property PREFERRED_USERNAME is missing documentation. Warning

The property PREFERRED_USERNAME is missing documentation.

Check warning on line 5 in authorization/src/main/kotlin/org/modelix/authorization/KeycloakTokenConstants.kt

See this annotation in the file changed.

Code scanning / detekt

PREFERRED_USERNAME can be a const val. Warning

PREFERRED_USERNAME can be a const val.

Check warning on line 6 in authorization/src/main/kotlin/org/modelix/authorization/KeycloakTokenConstants.kt

See this annotation in the file changed.

Code scanning / detekt

The property REALM_ACCESS is missing documentation. Warning

The property REALM_ACCESS is missing documentation.