Skip to content

Conversation

@ajjindal
Copy link
Contributor

  • Supports Tracing
  • Captures function payload with the send-default-pii

@vercel
Copy link

vercel bot commented Sep 29, 2020

This pull request is being automatically deployed with Vercel (learn more).
To see the status of your deployment, click below or on the icon next to each commit.

🔍 Inspect: https://vercel.com/sentry/sentry-docs/4n6mometq/sentry.dev
✅ Preview: https://sentry-docs-git-feat-serverlesstransactionpython.sentry.dev

Copy link
Contributor

@PeloWriter PeloWriter left a comment

Choose a reason for hiding this comment

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

LGTM


<Alert level="info" title="Note"><markdown>

If you are using another web framework inside of AWS Lambda, the framework might catch those exceptions before we get to see them. Make sure to enable the framework specific integration as well, if one exists. See [_Integrations_](/platforms/python/#integrations) for more information.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggestion to clarify:

Suggested change
If you are using another web framework inside of AWS Lambda, the framework might catch those exceptions before we get to see them. Make sure to enable the framework specific integration as well, if one exists. See [_Integrations_](/platforms/python/#integrations) for more information.
If you are using another web framework inside of AWS Lambda, the framework might catch those exceptions before they are sent to Sentry. As a result, we recommend enabling the framework-specific integration as well, if one exists. See [_Integrations_](/platforms/python/#integrations) for more information.


- Automatically report all exceptions from your Lambda Functions
- Automatically report all exceptions and transactions from your Lambda Functions
- You can control transactions sample rate using `traces_sample_rate`. You can learn more about transactions in [Distributed Tracing](/performance-monitoring/distributed-tracing/#traces-transactions-and-spans)
Copy link
Contributor

Choose a reason for hiding this comment

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

Note - the content for this setting is forthcoming in PR 2376.

Copy link
Member

Choose a reason for hiding this comment

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

ok let's not merge this PR before that one then. It seems that even with the current version of #2376 this will 404, so we'll have to re-review.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I took in the latest and the link still looks valid. @PeloWriter do you want me to update anything specific?

Copy link
Contributor

@PeloWriter PeloWriter Sep 30, 2020

Choose a reason for hiding this comment

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

This link may be more helpful. So perhaps edit to:

@vercel vercel bot temporarily deployed to Preview September 29, 2020 22:15 Inactive

- Automatically report all exceptions from your Lambda Functions
- Automatically report all exceptions and transactions from your Lambda Functions
- You can control transactions sample rate using `traces_sample_rate`. You can learn more about transactions in [Distributed Tracing](/performance-monitoring/distributed-tracing/#traces-transactions-and-spans)
Copy link
Member

Choose a reason for hiding this comment

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

ok let's not merge this PR before that one then. It seems that even with the current version of #2376 this will 404, so we'll have to re-review.

dcramer and others added 11 commits September 30, 2020 13:20
Docs has many broken/dead fragment URLs because an anchored section of a page moved or was renamed.
)

* fix(urls): Use relative URLs instead of linking to docs.sentry.io

* Revert relative URL change to api docs link

* Fix grouping & fingerprinting link

* Fix broken url on log4net docs
Co-authored-by: Fiona <[email protected]>
Co-authored-by: Bruno Garcia <[email protected]>
Co-authored-by: Manoel Aranda Neto <[email protected]>
dcramer and others added 6 commits September 30, 2020 16:02
- Expose dropdown for Product navigation
- Move all product pages into /product/
- Re-order sidebar to push relevant (vs marketing/discovery) docs to the top
…ocs (#2376)

Also a bunch of reorganization, and reworking of some (but not all) of the JS performance docs.
@untitaker
Copy link
Member

@ajjindal the rebase is messed up, this PR now contains too many unrelated changes to be reviewable

@ajjindal ajjindal closed this Oct 1, 2020
@ajjindal
Copy link
Contributor Author

ajjindal commented Oct 1, 2020

Creating a different PR

@ajjindal ajjindal deleted the feat/serverlessTransactionPython branch October 1, 2020 18:12
@github-actions github-actions bot locked and limited conversation to collaborators Dec 16, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.