-
Notifications
You must be signed in to change notification settings - Fork 156
feat(metrics): allow setting functionName via constructor parameter and environment variable #3696
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
Merged
dreamorosi
merged 21 commits into
aws-powertools:main
from
steven10172:improv/metrics-no-override-function-name
Mar 20, 2025
Merged
Changes from 18 commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
185e15b
feat(metrics): add ability to pass functionName to middy and decorator
steven10172 ff79706
docs(metrics): add setting function name section
steven10172 27875ee
docs(metrics): updated highlighted line for setting function name dec…
steven10172 68439d6
refactor(metrics): revert setting functionName in middy and decorator…
steven10172 9efee79
feat(metrics): allow setting functionName via ENV and constructor par…
steven10172 4de0264
docs(metrics): update docs and mention functionName constructor param…
steven10172 0a2cec8
Merge branch 'aws-powertools:main' into improv/metrics-no-override-fu…
steven10172 0616dc5
docs(metrics): add reference to POWERTOOLS_METRICS_FUNCTION_NAME on h…
steven10172 1c4c25a
refactor(metrics): cleanup code based on sonarqubecloud
steven10172 2c81643
refactor(metrics): deprecated setFunctionName and expand captureColdS…
steven10172 1c099b4
docs(metrics): update to become more inline with implementation
steven10172 61cfd8c
doc updates
steven10172 55ee400
Merge branch 'aws-powertools:main' into improv/metrics-no-override-fu…
steven10172 2378cb2
more doc updates
steven10172 6b008c2
Merge branch 'main' into improv/metrics-no-override-function-name
dreamorosi f5e5ac0
chore: align with suggested implementation
dreamorosi 21dd0c8
chore: ignore deprecated method from coverage
dreamorosi 29a6280
chore: format table
dreamorosi d69c2a0
chore: address review comments
dreamorosi 9631937
chore: address sonar
dreamorosi 258becd
Merge branch 'main' into improv/metrics-no-override-function-name
dreamorosi File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or 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 file contains hidden or 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 file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,17 @@ | ||
import { MetricUnit, Metrics } from '@aws-lambda-powertools/metrics'; | ||
|
||
const metrics = new Metrics({ | ||
namespace: 'serverlessAirline', | ||
serviceName: 'orders', | ||
}); | ||
|
||
export const handler = async ( | ||
_event: unknown, | ||
_context: unknown | ||
): Promise<void> => { | ||
metrics.captureColdStartMetric('my-function-name'); | ||
|
||
metrics.addMetric('successfulBooking', MetricUnit.Count, 1); | ||
|
||
metrics.publishStoredMetrics(); | ||
}; |
This file contains hidden or 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,18 @@ | ||
import { MetricUnit, Metrics } from '@aws-lambda-powertools/metrics'; | ||
|
||
const metrics = new Metrics({ | ||
namespace: 'serverlessAirline', | ||
serviceName: 'orders', | ||
functionName: 'my-function-name', | ||
}); | ||
|
||
export const handler = async ( | ||
_event: unknown, | ||
_context: unknown | ||
): Promise<void> => { | ||
metrics.captureColdStartMetric(); | ||
|
||
metrics.addMetric('successfulBooking', MetricUnit.Count, 1); | ||
|
||
metrics.publishStoredMetrics(); | ||
}; |
This file contains hidden or 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 file contains hidden or 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 file contains hidden or 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 file contains hidden or 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
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.