Comments (5)
pankajagrawal16
commented on August 17, 2024
Hi @bbatpp Thanks for reporting the issue. I will have a look at this.
from powertools-lambda-java.
pankajagrawal16
commented on August 17, 2024
Hi @bbatpp I tried replicating the issue and it seems to work fine in the sample application that I have. Meaning .isDebugEnabled return correct value.
That being said, It is a best practice to set
getContexttofalsein application server environment where different applications and even the application server itself might be using log4j2. Per my understanding, setting it to true one might end up re-configuring/writing into loggers of other applications or application server depending on how they in turn have obtained the context.
Could it be an idea for you to just try what the behavior is when you set it to true in your application?
from powertools-lambda-java.
pankajagrawal16
commented on August 17, 2024
Hi @bbatpp Any more insights into this issue?
from powertools-lambda-java.
bbatpp
commented on August 17, 2024
I changed where we instantiate the logger to pass true instead of false.
LoggerContext ctx = LogManager.getContext(cls.getClassLoader(), true);
This resolved the issue, I think what was happening is, that my lambda was
connected to the parent context. So when you changed the log level with the
lambda's environment variable, it was actually affecting the parent context
(AWS), rather than my lambda.
…On Wed, Dec 15, 2021 at 9:23 AM Pankaj Agrawal ***@***.***> wrote:
Hi @bbatpp <https://github.com/bbatpp> Any more insights into this issue?
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#551 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AQYI46KD7KLHPHFT6KPNYP3URCQG7ANCNFSM5ERL7K5Q>
.
Triage notifications on the go with GitHub Mobile for iOS
<https://apps.apple.com/app/apple-store/id1477376905?ct=notification-email&mt=8&pt=524675>
or Android
<https://play.google.com/store/apps/details?id=com.github.android&referrer=utm_campaign%3Dnotification-email%26utm_medium%3Demail%26utm_source%3Dgithub>.
--
Thanks,
Bryan Brown
SDE III, Fulfillment Automation
***@***.*** ***@***.***> • 207-956-3733
Amazon Pharmacy | Your medication, made easy
--
_The information contained in this transmission may contain privileged and
confidential information, including patient information protected by
federal and state privacy laws. It is intended only for the use of the
person(s) named above. If you are not the intended recipient, you are
hereby notified that any review, dissemination, distribution, or
duplication of this communication is strictly prohibited. If you are not
the intended recipient, please contact the sender by reply email and
destroy all copies of the original message._
from powertools-lambda-java.
pankajagrawal16
commented on August 17, 2024
Ok makes sense. In that case I will close the issue for now. Feel free to open new issue/reopen this in case of further issues. @bbatpp
from powertools-lambda-java.
Related Issues (20)
- Unexpected error occurred when using powertools in lambda function HOT 6
- Trace Annotations not getting added if Annotation Key has a space in it HOT 11
- Putting metrics fails when POWERTOOLS_METRICS_NAMESPACE is not set HOT 3
- Logging: Trace ID not logged with Java 17 runtime HOT 1
- Feature enhancement: Create a Subsegments for operations handled by Powertools HOT 5
- Logging: print `message` as JSON, instead of wrapping into a string HOT 14
- Exceptions not showing in Traces HOT 1
- RFC: Introduce Version 2
- RFC: Remove support for Java 8 in V2 HOT 7
- Feature enhancement (Logger): automatically intercept Runtime exceptions, and log information HOT 7
- Log level not restored at the end of execution HOT 2
- Support batch secrets retrieval in Parameters module HOT 6
- Maintenance: update jackson and aws-xray-recorder dependencies together HOT 4
- RFC: handle batch messages in parallel in batch module HOT 13
- v2: Example logging configuration needs to be updated HOT 3
- v2 - Params builders do not supply default TransformationManager HOT 1
- bug - v2 - end-to-end tests broken
- AbstractCustomResourceHandler - Unable to send response HOT 2
- LambdaEcsLayout seems to incorrectly serialize the service field HOT 7
- Add support for CRaC
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from powertools-lambda-java.