Skip to content

Feature implementation from commits 0ba0ac8..aa0b3cb#2

Open
yashuatla wants to merge 15 commits intofeature-base-2from
feature-head-2
Open

Feature implementation from commits 0ba0ac8..aa0b3cb#2
yashuatla wants to merge 15 commits intofeature-base-2from
feature-head-2

Conversation

@yashuatla
Copy link
Copy Markdown
Owner

This PR contains changes from a range of commits from the original repository.

Commit Range: 0ba0ac8..aa0b3cb
Files Changed: 111 (94 programming files)
Programming Ratio: 84.7%

Commits included:

kimlisa and others added 15 commits June 23, 2025 18:10
…vitational#56002)

On Linux, docker buildx plugin doesn't automatically load images into
the registry so it never rebuilds the image on go version changes or buf
changes. This causes buf to fail because the image go base version
differs from the Teleport required version.

Since we do not need any cross-compilation here, we can safely drop
`buildx` plugin usage.

Signed-off-by: Tiago Silva <tiago.silva@goteleport.com>
* MWI: Add audit events for bound keypair joining

This adds 3 new audit events for bound keypair joining:
- `join_token.bound_keypair.recovery` - emitted when a join triggers
  a recovery (first join, or join with expired certs)
- `join_token.bound_keypair.rotation` - emitted when a keypair
  rotation takes place
- `join_token.bound_keypair.join_state_verification_failed` - emitted
  when the client provides an invalid join state document

* Fix UI lint

* Fix more UI lints

* Remove outdated TODO

* Fix tests broken by error message changes
gravitational#56016)

When building on Linux, often git fails to switch to between branches because generated files aren't owned by the user.
This PR fixes that by injecting the UID and GID into the docker container
* feat: Add bot instance cache

* Finalise cache implementation and add tests

* Appease the linter

* Return early in paging logic

* Fix import ordering/grouping

* Fix test cache setup

* Revert BotCache rename

* Clean-up commented code

* Revert imports order change

* Refactor to use generic getter and lister
```
 panic: runtime error: invalid memory address or nil pointer dereference
[signal SIGSEGV: segmentation violation code=0x1 addr=0x20 pc=0x5af856fc3494]

goroutine 62984 [running]:
github.com/gravitational/teleport/lib/backend/firestore.(*Backend).Items.func3-range1(0xc0001991f0, {0x0?, 0x0?})
        github.com/gravitational/teleport/lib/backend/firestore/firestorebk.go:586 +0x134
github.com/gravitational/teleport/lib/backend/firestore.(*Backend).Items.func3.(*Backend).mergedRecords.2-range1(0x5af85ed273a0?, {0x0, 0x0?})
        github.com/gravitational/teleport/lib/backend/firestore/firestorebk.go:655 +0x4f
github.com/gravitational/teleport/lib/itertools/stream.MergeStreams[...].func1()
        github.com/gravitational/teleport/lib/itertools/stream/stream.go:378 +0x236
github.com/gravitational/teleport/lib/backend/firestore.(*Backend).Items.func3.(*Backend).mergedRecords.2(0xc000e52500)
        github.com/gravitational/teleport/lib/backend/firestore/firestorebk.go:654 +0x89b
github.com/gravitational/teleport/lib/backend/firestore.(*Backend).Items.func3(0xc003bb4cc0)
        github.com/gravitational/teleport/lib/backend/firestore/firestorebk.go:579 +0x3bf
```

The above panic is caused by the snapShot not being set and thus
resulting in a nil dereference. The snapShot is now assigned outside
of the switch to ensure that all records that are successfully
created contain a reference to a valid DocumentSnapshot.
…nal#55433)

* Add support for rocky and almalinux to CDN install script

Extends gravitational#44171

* Add rocky and alma as search keywords on discover page

* Add rocky and alma as search keywords on discover page

* Fix prettier

* Fix prettier
* Add info to tsh tests

* Specify session id
…al#55693)

We had been updating the client activity tracker any time a message
was sent from the client (browser or Teleport Connect) to the remote
host. This approach was fine for the original RDP implmentation, as
all messages sent in this direction were for user input (keypresses,
mouse movement, scroll wheel, etc), but it is insufficient with the
current RemoteFX implementation as there are some messages sent by
the remote Windows hosts which require client acknowledgements.
These acknowledgements were mistakenly being counted as client activity.

Closes gravitational#55691
…mplete (gravitational#56020)

* [vnet] fix: close proxied channel after source channel terminates

* buffer error chan, avoid unecessary error wrap
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.