Age | Commit message (Collapse) | Author | |
---|---|---|---|
2023-04-12 | Update golangci-lint tool | Kassio Borges | |
2022-07-05 | Remove rate limits feature flags | feistel | |
Changelog: removed | |||
2022-06-15 | Remove host out of Artifact.TryMakeRequest | feistel | |
2022-06-05 | Refactor acme tests and middleware for lazy domain resolution | feistel | |
2022-05-05 | Abstract artifact handling to a separate middleware | feistel | |
2022-05-02 | Add nolintlint linter configuration | feistel | |
Require an explanation for nolint rules and warn about unused rules | |||
2022-04-26 | Merge branch 'refactor/t-setenv' into 'master' | Jaime Martinez | |
refactor: replace os.Setenv with testing.SetEnv Closes #676 See merge request gitlab-org/gitlab-pages!742 | |||
2022-04-26 | Rework HTTPS tests and assert location header | feistel | |
2022-04-25 | Add early return and tests for internal/handlers/https | feistel | |
2022-04-25 | Replace os.Setenv with testing.SetEnv | feistel | |
2022-04-22 | Move https middleware to internal/handlers | feistel | |
2022-02-24 | test: replace require.Equal with specialized assertions | feistel | |
2022-02-21 | feat: Add TLS rate limits | Vladimir Shushlin | |
Changelog: added | |||
2022-01-26 | test: move mocks to their own package | feistel | |
2022-01-24 | lint: fix gci issues | feistel | |
2022-01-11 | feat: add domain rate-limiter | Vladimir Shushlin | |
Changelog: added | |||
2021-12-20 | refactor: handle defaults in ratelimiter package itself | Vladimir Shushlin | |
also fix tests: * float64(1/time.Milesecond) == 0 * rate package doesn't actually refill the bucket on fractional seconds, so we need to use integers | |||
2021-12-13 | refactor: move ratelimiter configuration to handlers package | Vladimir Shushlin | |
2021-11-22 | lint: remove outdated nolint directives | feistel | |
2021-11-17 | test: stop calling mockController.Finish directly | feistel | |
This is handled by mockgen 1.5.0+ | |||
2021-09-19 | chore: clean up handler tests to use httptest.NewRequest | Stan Hu | |
2021-09-19 | fix: handle 403 errors when artifacts loaded from GitLab artifacts browser | Stan Hu | |
When the artifacts server is enabled, GitLab Rails links certain files (e.g. HTML) to the GitLab Pages server so that the file can be rendered instead of downloaded. However, if public pipelines are not enabled for a public project and the user never authenticated with the Pages server, the GitLab Artifacts API would return a 403 error. This error would be forwarded to the user, and Pages would halt further processing. To fix this problem, when we encounter a 403 error, we attempt to authenticate the user if no token is available. Relates to https://gitlab.com/gitlab-org/gitlab/-/issues/25192 Changelog: fixed | |||
2021-08-19 | docs: add related issue to TODO comment | feistel | |
2021-08-19 | test: fix response body not being closed | feistel | |
nolint is added when the body is nil or if the body can't be closed | |||
2020-06-01 | Enable the remaining linters | Jaime Martinez | |
- ineffassign - misspell - structcheck - typecheck - unconvert - unused - varcheck - whitespace | |||
2020-06-01 | Enable bodyclose linter | Jaime Martinez | |
2019-09-26 | Add support for previewing artifacts that are not public | Tuomo Ala-Vannesluoma | |
Remove some duplicate logic on Auth module Separate handling artifact to own handlers package Unit test handlers by mocking auth and artifact modules Add generate-mock step to Makefile Use additional handler func to simplify TryMakeRequest return type Always try with token if exists Do not log RequestURI, log path only Remove not used logRequest func |