feat(authz): Subject mapping OPA builtin for condition evaluation and jq selection #2015
elizabethhealychecks.yaml
on: pull_request
Matrix: go
Matrix: integration tests
Validate PR title
3s
image build
1m 36s
Protocol Buffer Lint and Gencode Up-to-date check
24s
license check
30s
ci
0s
Annotations
17 errors and 13 warnings
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L24
commentFormatting: put a space between `//` and comment text (gocritic)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L113
ifElseChain: rewrite if-else to switch statement (gocritic)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L125
ifElseChain: rewrite if-else to switch statement (gocritic)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L153
ifElseChain: rewrite if-else to switch statement (gocritic)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L63
avoid direct access to proto field mapping.Value.SubjectMappings, use mapping.GetValue().GetSubjectMappings() instead (protogetter)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L65
avoid direct access to proto field subject_mapping.SubjectConditionSet.SubjectSets, use subject_mapping.GetSubjectConditionSet().GetSubjectSets() instead (protogetter)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L94
avoid direct access to proto field subject_set.ConditionGroups, use subject_set.GetConditionGroups() instead (protogetter)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L115
avoid direct access to proto field condition_group.BooleanOperator, use condition_group.GetBooleanOperator() instead (protogetter)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L120
avoid direct access to proto field condition_group.Conditions, use condition_group.GetConditions() instead (protogetter)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L132
avoid direct access to proto field condition_group.BooleanOperator, use condition_group.GetBooleanOperator() instead (protogetter)
|
|
go (sdk)
The job was canceled because "service" failed.
|
|
go (sdk)
The operation was canceled.
|
|
integration tests (standard)
Error: Timed out waiting for: tcp:localhost:8080
|
|
integration tests (standard)
Timed out waiting for: tcp:localhost:8080
|
|
integration tests (hsm)
The job was canceled because "standard" failed.
|
|
integration tests (hsm)
The operation was canceled.
|
|
ci
Process completed with exit code 1.
|
|
Protocol Buffer Lint and Gencode Up-to-date check
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: bufbuild/buf-setup-action@382440cdb8ec7bc25a68d7b4711163d95f7cc3aa, bufbuild/buf-lint-action@044d13acb1f155179c606aaa2e53aea304d22058, bufbuild/buf-breaking-action@a074e988ee34efcd4927079e79c611f428354c01. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|
|
Protocol Buffer Lint and Gencode Up-to-date check
No github_token supplied, API requests will be subject to stricter rate limiting
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L59
var-declaration: should omit type []string from declaration of var entitlements; it will be inferred from the right-hand side (revive)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L117
empty-block: this block is empty, you can remove it (revive)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L139
empty-block: this block is empty, you can remove it (revive)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L188
empty-block: this block is empty, you can remove it (revive)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L25
var-naming: don't use underscores in Go names; var attribute_mappings should be attributeMappings (revive)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L26
var-naming: don't use underscores in Go names; var entity_representations should be entityRepresentations (revive)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L56
var-naming: don't use underscores in Go names; func parameter attribute_mappings should be attributeMappings (revive)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L62
var-naming: don't use underscores in Go names; var mapping_result should be mappingResult (revive)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L64
var-naming: don't use underscores in Go names; var subject_mapping_result should be subjectMappingResult (revive)
|
|
go (service):
service/internal/subjectmappingbuiltin/subject_mapping_builtin.go#L66
var-naming: don't use underscores in Go names; var subject_set_condition_result should be subjectSetConditionResult (revive)
|
|
integration tests (standard)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: JarvusInnovations/background-action@313d37130873d82c33fc907b9b78e932aec8e990. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
|