-
Notifications
You must be signed in to change notification settings - Fork 290
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Backport of docs: Fix broken link into stable-website #4036
Merged
hc-github-team-secure-boundary
merged 28 commits into
stable-website
from
backport/dheath-fix-host-discovery-link/arguably-enjoyed-martin
Nov 13, 2023
Merged
Backport of docs: Fix broken link into stable-website #4036
hc-github-team-secure-boundary
merged 28 commits into
stable-website
from
backport/dheath-fix-host-discovery-link/arguably-enjoyed-martin
Nov 13, 2023
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This fixes test failures introduced in a recent refactoring. Blame: 0bf6099
This reverts commit 274aca7.
* chore: Update changelog for 0.14.2 release
Two Vault client settings were not being properly used when constructing a Vault client. The `TLS Skip Verify` setting was only being set if a `CA Cert` was also configured. This fix sets the `TLS Skip Verify` when configured regardless of other settings. Closes #3961. The `TLS Server Name` setting was never being set. Bad programmers. This fix now sets it on the Vault client if the Vault Credential Store has been configured to use a value for this setting. Closes #3962.
* docs: Add enterprise badge * docs: Add heading-level enterprise alert * docs: Add some missing tags
The nhooyr/websocket library now has a release with the new default behavior that removes the message size limit. This allows us to switch off of the fork and back to the upstream. In the same release that includes the fix we need for the boundary client, the wspb helper package was removed from websocket. This package contained some functions to read and write protobuf messages. However, it was using the deprecated github.com/golang/protobuf/proto library. To avoid this dependancy in the library they opted to just remove wspb. Thus this provides a replacement implementation that uses google.golang.org/protobuf/proto. Notably the newer proto library seems to lack an equivalent of proto.NewBuffer so writes to the websocket and not able to leverage a sync.Pool of byte.Buffers to reduce allocations. Refs: 88bb253 See: coder/websocket@ac38512
* Minor test fixes The auth rotation test fix allows us to keep checking if we did not yet see the expected bytes in case the call to lookup succeeds but we are late in rotating due to parallelism. The api credentials test fix allows us to check if at _least_ one rotation has happened. That way if two rotations happen before we check again due to parallelism, this is deemed a successful result. * Add timeout to auth rotation test
* If passing a private key without specifying it in URL syntax, give a proper error message (previously we were only making sure it was a scheme we knew how to handle, but we need to also be more informative if it's not a URL at all) * If the handshake from the proxy fails, don't hang, just exit * Update some error messages to use capitalization in line with our standard CLI error format
* refact(e2e): Create module to run boundary commands * test(e2e): Add worker-led registration test
* Update LICENSE * make line wrap more consistent
30e2541
to
a892470
Compare
Database schema diff between To understand how these diffs are generated and some limitations see the FunctionsUnchanged TablesUnchanged ViewsUnchanged TriggersUnchanged IndexesUnchanged ConstraintsUnchanged Foreign Key ConstraintsUnchanged |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Backport
This PR is auto-generated from #4035 to be assessed for backporting due to the inclusion of the label backport/website.
The below text is copied from the body of the original PR.
There is a broken link in the Host discovery index page. It should go to the AWS host catalogs tutorial:
This PR updates the link. I also fixed a recently introduced typo on the Validate the integrity of session recordings page.
View the preview deployment:
UPDATE: I found and fixed 2 more broken links in the same topic here: