-
Notifications
You must be signed in to change notification settings - Fork 144
What information does Flask-SQLAlchemy send? #33
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
Comments
The actual query string capture was originally in the PR submitted by @therealryanbonham, but we requested that part of logic to be removed and submitted with a separate PR. The actual query might contain sensitive user information and we will have to require the query capture PR to be sanitized and reviewed by our security team to make sure no sensitive user data can sneak into the captured data. We haven't receive the separate PR yet but I will ping @therealryanbonham to check if there is any recent plan on the query capture. |
can you link me to that PR ? |
Yes sorry I forget to put that link on. #14 |
Hi @harveyslash just want to let you know that the PR for query capture is submitted: #34. Please feel free to try it out and provide any feedback under that PR. |
I installed that using: And it doesnt seem to have changed anything. I am not getting the actual query in my X-ray console |
You need to include the branch.. |
just tried the branch, now im getting :
And my X-ray console doesnt show the requests that had the message too long error. |
The default UDP packet size is usually 64KB but it might vary on different OS. You can consider to increase this limit. You can also change See https://github.com/aws/aws-xray-sdk-python/blob/master/aws_xray_sdk/core/recorder.py#L118. The default value is 30 which means a segment will stream out its children subsegments when the total children count reaches 30. You can change to 10 or even lower to fit your use case. |
Since the PR for query capture is already out #34, I will close this in favor of the PR to make sure we have a single place to track feedback, concerns and progress regarding this new feature. If you have other questions regarding the SDK usage in general, feel free to post on our forum https://forums.aws.amazon.com/forum.jspa?forumID=241 or open new issues. But feel free to leave any comments here if you still have concerns. |
I would like to get the actual queries that are being executed along with the time it took to execute them.
Right now the most informative text that I can see is
sqlalchemy.orm.query.one
(indicating that i performed a one() query)Is there any way to get actual sql query data , or the queries itself ?
The text was updated successfully, but these errors were encountered: