From a118c6ce8df9411372512c3addd5e8b718ea1ae0 Mon Sep 17 00:00:00 2001 From: Johan Brandhorst-Satzkorn Date: Fri, 6 Sep 2024 15:43:45 -0700 Subject: [PATCH 1/3] internal/session: update variable name in test This was missed in #5071 --- internal/session/job_session_cleanup_test.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/session/job_session_cleanup_test.go b/internal/session/job_session_cleanup_test.go index c98459778f..3e7e8a5284 100644 --- a/internal/session/job_session_cleanup_test.go +++ b/internal/session/job_session_cleanup_test.go @@ -95,7 +95,7 @@ func TestSessionConnectionCleanupJob(t *testing.T) { // Create the job. job, err := newSessionConnectionCleanupJob(ctx, rw, gracePeriod) - job.gracePeriod = gracePeriod // by-pass factory assert so we dont have to wait so long + job.workerStatusGracePeriod = gracePeriod // by-pass factory assert so we dont have to wait so long require.NoError(err) // sleep the status grace period. From e833d79fc580bf76c48063301cffe4ae71f08031 Mon Sep 17 00:00:00 2001 From: Johan Brandhorst-Satzkorn Date: Fri, 6 Sep 2024 16:12:49 -0700 Subject: [PATCH 2/3] internal/gen: regenerate files --- internal/gen/controller.swagger.json | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/internal/gen/controller.swagger.json b/internal/gen/controller.swagger.json index b6d9321476..40e694ab4a 100644 --- a/internal/gen/controller.swagger.json +++ b/internal/gen/controller.swagger.json @@ -3,7 +3,7 @@ "info": { "title": "Boundary controller HTTP API", "description": "Welcome to the Boundary controller HTTP API documentation. This page provides a reference guide for using the Boundary controller API, a JSON-based HTTP API. The API implements commonly seen HTTP API patterns for status codes, paths, and errors. See the [API overview](https://developer.hashicorp.com/boundary/docs/api-clients/api) for more information.\n\nBefore you read this page, it is useful to understand Boundary's [domain model](https://developer.hashicorp.com/boundary/docs/concepts/domain-model) and to be aware of the terminology used here. To get started, search for the service you want to interact with in the sidebar to the left. Each resource in Boundary, such as accounts and credential stores, has its own service. Each service contains all the API endpoints for the resource.\n## Status codes\n- `2XX`: Boundary returns a code between `200` and `299` on success. Generally this is `200`, but implementations should be prepared to accept any `2XX` status code as indicating success. If a call returns a `2XX` code that is not `200`, it follows well-understood semantics for those status codes.\n- `400`: Boundary returns `400` when a command cannot be completed due to invalid user input, except for a properly-formatted identifier that does not map to an existing resource, which returns a `404` as discussed below.\n- `401`: Boundary returns `401` if no authentication token is provided or if the provided token is invalid. A valid token that simply does not have permission for a resource returns a `403` instead. A token that is invalid or missing, but where the anonymous user (`u_anon`) is able to successfully perform the action, will not return a `401` but instead will return the result of the action.\n- `403`: Boundary returns `403` if a provided token was valid but does not have the grants required to perform the requested action.\n- `404`: Boundary returns `404` if a resource cannot be found. Note that this happens _prior_ to authentication/authorization checking in nearly all cases as the resource information (such as its scope, available actions, etc.) is a required part of that check. As a result, an action against a resource that does not exist returns a `404` instead of a `401` or `403`. While this could be considered an information leak, since IDs are randomly generated and this only discloses whether an ID is valid, it's tolerable as it allows for far simpler and more robust client implementation.\n- `405`: Boundary returns a `405` to indicate that the method (HTTP verb or custom action) is not implemented for the given resource.\n- `429`: Boundary returns a `429` if any of the API rate limit quotas have been exhausted for the resource and action. It includes the `Retry-After` header so that the client knows how long to wait before making a new request.\n- `500`: Boundary returns `500` if an error occurred that is not (directly) tied to invalid user input. If a `500` is generated, information about the error is logged to Boundary's server log but is not generally provided to the client.\n- `503`: Boundary returns a `503` if it is unable to store a quota due to the API rate limit being exceeded. It includes the `Retry-After` header so that the client knows how long to wait before making a new request.\n## List pagination\nBoundary uses [API pagination](https://developer.hashicorp.com/boundary/docs/api-clients/api/pagination) to support searching and filtering large lists of results efficiently.", - "version": "0.17.1", + "version": "0.17.2", "contact": { "name": "HashiCorp Boundary", "url": "https://www.boundaryproject.io/" From cece7dbdc2428bbab8d250707316766cfaa587c1 Mon Sep 17 00:00:00 2001 From: Johan Brandhorst-Satzkorn Date: Fri, 6 Sep 2024 17:07:02 -0700 Subject: [PATCH 3/3] internal/host/plugin: fix flaky test Hosts contain IP addresses in string form, whose order is not defined. --- internal/host/plugin/repository_host_test.go | 9 +++++++++ 1 file changed, 9 insertions(+) diff --git a/internal/host/plugin/repository_host_test.go b/internal/host/plugin/repository_host_test.go index 1f9f72a686..e2caef7f2c 100644 --- a/internal/host/plugin/repository_host_test.go +++ b/internal/host/plugin/repository_host_test.go @@ -221,6 +221,9 @@ func TestJob_UpsertHosts(t *testing.T) { cmpopts.SortSlices(func(x, y *Host) bool { return x.GetPublicId() < y.GetPublicId() }), + cmpopts.SortSlices(func(x, y string) bool { + return x < y + }), ), ) @@ -242,6 +245,9 @@ func TestJob_UpsertHosts(t *testing.T) { cmpopts.SortSlices(func(x, y *Host) bool { return x.GetPublicId() < y.GetPublicId() }), + cmpopts.SortSlices(func(x, y string) bool { + return x < y + }), ), ) plg := plg @@ -274,6 +280,9 @@ func TestJob_UpsertHosts(t *testing.T) { got, cmpopts.IgnoreUnexported(Host{}, store.Host{}), cmpopts.IgnoreTypes(×tamp.Timestamp{}), + cmpopts.SortSlices(func(x, y string) bool { + return x < y + }), ), ) assert.Empty(