Better link of aws cloudwatch logs.

[5achinJani]

• Review the documentation: https://docs.sentry.io/
• Search for existing issues: https://github.com/getsentry/sentry-javascript/issues
• Use the latest release: https://github.com/getsentry/sentry-javascript/releases
• Provide a link to the affected event from your Sentry account

Package + Version

• @sentry/browser
• @sentry/node
• raven-js
• raven-node (raven for node)
• @sentry/serverless

Version:

^6.10.0

Description

Is it possible that url of AWS CLOUDWATCH LOGS that sentry serverless attaches opens up directly the filtered logs showing the exact logs of the aws_request_id ? This would help us a lot. Currently we've to apply the time slot and aws_request_id search filter after opening the cloudwatch logs page from sentry issue.

[AbhiPrasad]

Hey, thanks for writing in. We've add this serverless improvement to the backlog, but in the meantime if you (or anybody else) are interested in contributing, PRs are welcome!

We set the aws context here:

sentry-javascript/packages/serverless/src/awslambda.ts

Lines 161 to 179 in 6ae95a4

	scope.setContext('aws.lambda', {
	aws_request_id: context.awsRequestId,
	function_name: context.functionName,
	function_version: context.functionVersion,
	invoked_function_arn: context.invokedFunctionArn,
	execution_duration_in_millis: performance.now() - startTime,
	remaining_time_in_millis: tryGetRemainingTimeInMillis(context),
	'sys.argv': process.argv,
	});
	scope.setContext('aws.cloudwatch.logs', {
	log_group: context.logGroupName,
	log_stream: context.logStreamName,
	url: `https://console.aws.amazon.com/cloudwatch/home?region=${
	process.env.AWS_REGION
	}#logsV2:log-groups/log-group/${encodeURIComponent(context.logGroupName)}/log-events/${encodeURIComponent(
	context.logStreamName,
	)}`,
	});

[tmilar]

Hi @AbhiPrasad , I've been wishing for the same feature for some time now, and found it that it was a surprisingly easy change.
I've submitted a PR #4032 😄

[AbhiPrasad]

Thanks for your PR @tmilar. #4032 has merged into master and will be available in the next release of the SDK. Cheers!