This repository has been archived by the owner on Nov 14, 2024. It is now read-only.
Url preview endpoint #3958
Triggered via pull request
September 30, 2024 11:28
adnull
Status
Failure
Total duration
4m 52s
Artifacts
–
dendrite.yml
on: pull_request
Matrix: complement
Matrix: sytest
Integration tests
0s
Upgrade tests
0s
Upgrade tests from HEAD-2
0s
Update Docker images
/
Monolith image
Update Docker images
/
Pinecone demo image
Update Docker images
/
Yggdrasil demo image
Annotations
30 errors and 7 warnings
|
Build for Windows (windows, amd64)
Cannot open: File exists
|
|
Build for Windows (windows, amd64)
Cannot open: File exists
|
|
Build for Windows (windows, amd64)
Cannot open: File exists
|
|
Build for Windows (windows, amd64)
Cannot open: File exists
|
|
Build for Windows (windows, amd64)
Cannot open: File exists
|
|
Build for Windows (windows, amd64)
Cannot open: File exists
|
|
Build for Windows (windows, amd64)
Cannot open: File exists
|
|
Build for Windows (windows, amd64)
Cannot open: File exists
|
|
Build for Windows (windows, amd64)
Cannot open: File exists
|
|
Build for Windows (windows, amd64)
Cannot open: File exists
|
|
Unit tests
Cannot open: File exists
|
|
Unit tests
Cannot open: File exists
|
|
Unit tests
Cannot open: File exists
|
|
Unit tests
Cannot open: File exists
|
|
Unit tests
Cannot open: File exists
|
|
Unit tests
Cannot open: File exists
|
|
Unit tests
Cannot open: File exists
|
|
Unit tests
Cannot open: File exists
|
|
Unit tests
Cannot open: File exists
|
|
Unit tests
Cannot open: File exists
|
|
Linting:
mediaapi/routing/url_preview.go#L175
Error return value of `resp.Body.Close` is not checked (errcheck)
|
|
Linting:
mediaapi/routing/url_preview.go#L359
Error return value of `file.Close` is not checked (errcheck)
|
|
Linting:
mediaapi/routing/url_preview.go#L42
cyclomatic complexity 26 of func `makeUrlPreviewHandler` is high (> 25) (gocyclo)
|
|
Linting:
mediaapi/routing/url_preview.go#L197
`orginal` is a misspelling of `original` (misspell)
|
|
Linting:
mediaapi/routing/url_preview.go#L260
getPreviewFromHTML - result 1 (error) is always nil (unparam)
|
|
Linting:
mediaapi/routing/url_preview.go#L514
`loadUrlPreviewResponse` - `logger` is unused (unparam)
|
|
Linting:
mediaapi/routing/url_preview.go#L340
shadow: declaration of "filename" shadows declaration at line 293 (govet)
|
|
Linting
issues found
|
|
Initial tests passed
Process completed with exit code 1.
|
|
Integration tests passed
Process completed with exit code 1.
|
|
Build for Linux (linux, amd64)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-go@v4. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Build for Windows (windows, amd64)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-go@v4. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Build for Windows (windows, amd64)
Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2
|
|
Build for Linux (linux, 386)
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-go@v4. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Unit tests
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-go@v4. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|
|
Unit tests
Failed to restore: "/usr/bin/tar" failed with error: The process '/usr/bin/tar' failed with exit code 2
|
|
Linting
The following actions use a deprecated Node.js version and will be forced to run on node20: actions/setup-go@v4, golangci/golangci-lint-action@v3. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
|