Age | Commit message (Collapse) | Author |
|
add Cache-Control: no-store to status response
|
|
Currently we do logging and metrics capturing
after we did the domain information lookup.
It allows us to add more information to access logs.
But it also distorts metrics because domain information
lookup takes time.
This logic was originally introduced in
https://gitlab.com/gitlab-org/gitlab-pages/-/merge_requests/157/diffs
It didn't matter back than because we didn't lookup domain via API as we do now.
Now it does matter.
So this commits moves metrics and logging middlewares almost to the top of pipeline.
Changelog: fixed
|
|
|
|
Changelog: added
|
|
|
|
|
|
|
|
disk source
|
|
when there are missing parameters such as `--listen-http`
by splitting configuration loading and validation.
Related to omnibus-gitlab#6321.
Changelog: other
|
|
defined as part of the configuration during daemon initialization.
Changelog: other
|
|
Uses the `internal/config/` client inside the `internal/source/gitlab/`
package which makes it easier to extend the configuration.
This is an iteration of https://gitlab.com/gitlab-org/gitlab-pages/-/issues/543.
Changelog: other
|
|
|
|
Signed-off-by: Balasankar "Balu" C <balasankarc@autistici.org>
|
|
Changelog: changed
Signed-off-by: Balasankar "Balu" C <balasankarc@autistici.org>
|
|
This reverts commit 185e0a0e5c621f350335495291c535e5c05df89b, reversing
changes made to a3365a7b1b41b3dee206cbcf27e915ee45d556a5.
|
|
This reverts merge request !305
|
|
Add more test cases for domains. Support sourceAuto and use IsReady
for gitlab source.
|
|
This way we short-circuit health check requests and avoid doing domain
lookup for them. We also do not report them in exported Prometheus
metrics and this way avoid trigger alerts during deploys.
Related to:
* https://gitlab.com/gitlab-org/gitlab-pages/issues/350
* https://gitlab.com/gitlab-com/gl-infra/production/issues/1681
|
|
them in the appropriate http.Request fields
|