Skip to content
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.

Sign up for GitHub

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

Jump to bottom

DRIVERS-2830 - FaaS detection logic mistakenly identifies EKS as AWS Lambda #1673

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

DRIVERS-2830 - FaaS detection logic mistakenly identifies EKS as AWS Lambda #1673

Changes from 4 commits
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
d890e0a
DRIVERS-2830 - FaaS detection logic mistakenly identifies EKS as AWS …
NoahStapp Oct 9, 2024
70eb2fc
Clarify test requirements
NoahStapp Oct 9, 2024
64ba5c1
Fix changelog date
NoahStapp Oct 9, 2024
56e6769
Update to reflect expected behavior
NoahStapp Oct 11, 2024
336b891
Merge branch 'master' into DRIVERS-2830
NoahStapp Oct 11, 2024
29fe44d
Add details to client.env
NoahStapp Oct 11, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 11 additions & 0 deletions source/mongodb-handshake/handshake.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -511,6 +511,16 @@ the following sets of environment variables:
| -------------------- | ----- |
| `AWS_EXECUTION_ENV` | `EC2` |

9. Valid container and FaaS provider. This test MUST verify that both the container metadata and the AWS Lambda metadata
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should include a note about this behavior in the client.env description above as well.

is present in `client.env`.

| Environment Variable | Value |
| --------------------------------- | ------------------ |
| `AWS_EXECUTION_ENV` | `AWS_Lambda_java8` |
| `AWS_REGION` | `us-east-2` |
| `AWS_LAMBDA_FUNCTION_MEMORY_SIZE` | `1024` |
| `KUBERNETES_SERVICE_HOST` | `1` |

## Motivation For Change

Being able to annotate individual connections with custom data will allow users and sysadmins to easily correlate events
Expand Down Expand Up @@ -593,6 +603,7 @@ support the `hello` command, the `helloOk: true` argument is ignored and the leg

## Changelog

- 2024-10-09: Clarify that FaaS and container metadata must both be populated when both are present.
- 2024-08-16: Migrated from reStructuredText to Markdown.
- 2019-11-13: Added section about supporting wrapping libraries
- 2020-02-12: Added section about speculative authentication
Expand Down
Loading