Skip to content

fix: Fixed subscriptions API filtering by topics #739

fix: Fixed subscriptions API filtering by topics

fix: Fixed subscriptions API filtering by topics #739

Triggered via pull request November 21, 2023 09:34
Status Failure
Total duration 5m 3s
Artifacts
This run and associated checks have been archived and are scheduled for deletion. Learn more about checks retention

ci.yml

on: pull_request
Matrix: Test
Fit to window
Zoom out
Zoom in

Annotations

10 errors
Lint: subscribe_test.go#L333
string `?topic=http://example.com/reviews/{id}` has 3 occurrences, make it a constant (goconst)
Lint: server_test.go#L358
string `/metrics` has 3 occurrences, make it a constant (goconst)
Lint: subscription.go#L36
string `/subscriptions` has 4 occurrences, make it a constant (goconst)
Lint: subscribe_test.go#L660
string `?topic=http://example.com/foos/{id}` has 4 occurrences, make it a constant (goconst)
Lint: subscription_test.go#L141
string `/subscriptions/` has 3 occurrences, make it a constant (goconst)
Lint: demo.go#L32
string `>; rel="mercure"` has 3 occurrences, make it a constant (goconst)
Lint: authorization.go#L57
string `*** has 31 occurrences, make it a constant (goconst)
Lint: server_test.go#L25
string `http://` has 9 occurrences, make it a constant (goconst)
Lint: authorization_test.go#L32
G101: Potential hardcoded credentials: RSA private key (gosec)
Lint: common/version_test.go#L98
avoid direct access to proto field metricOut.Gauge.Value, use metricOut.GetGauge().GetValue() instead (protogetter)