Skip to content

Feat: usage stats for screenshot generated#10706

Merged
lohanidamodar merged 1 commit intofeat-screenshots-endpointfrom
feat-screenshot-endpoint-stats
Oct 27, 2025
Merged

Feat: usage stats for screenshot generated#10706
lohanidamodar merged 1 commit intofeat-screenshots-endpointfrom
feat-screenshot-endpoint-stats

Conversation

@lohanidamodar
Copy link
Copy Markdown
Member

@lohanidamodar lohanidamodar commented Oct 27, 2025 •
edited
Loading

What does this PR do?

  • Usage stats for screenshots generated
  • Doesn't count on cache hit
image

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Screenshots may also be helpful.)

Related PRs and Issues

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Oct 27, 2025

Important

Review skipped

Auto reviews are disabled on base/target branches other than the default branch.

🗂️ Base branches to auto review (4)
  • main
  • 1.6.x
  • 1.7.x
  • 1.8.x

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch feat-screenshot-endpoint-stats

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 27, 2025

Security Scan Results for PR

Docker Image Scan Results

Package Version Vulnerability Severity
binutils 2.44-r2 CVE-2025-5244 HIGH
binutils 2.44-r2 CVE-2025-5245 HIGH
libxml2 2.13.8-r0 CVE-2025-49794 CRITICAL
libxml2 2.13.8-r0 CVE-2025-49796 CRITICAL
libxml2 2.13.8-r0 CVE-2025-49795 HIGH
libxml2 2.13.8-r0 CVE-2025-6021 HIGH
pcre2 10.43-r1 CVE-2025-58050 CRITICAL
golang.org/x/crypto v0.31.0 CVE-2025-22869 HIGH
golang.org/x/oauth2 v0.24.0 CVE-2025-22868 HIGH
stdlib 1.22.10 CVE-2025-47907 HIGH

Source Code Scan Results

🎉 No vulnerabilities found!

@github-actions
Copy link
Copy Markdown

github-actions bot commented Oct 27, 2025

✨ Benchmark results

  • Requests per second: 1,218
  • Requests with 200 status code: 219,302
  • P99 latency: 0.157949105

⚡ Benchmark Comparison

Metric This PR Latest version
RPS 1,218 1,233
200 219,302 222,033
P99 0.157949105 0.167493087

@lohanidamodar lohanidamodar merged commit 74bcfe1 into feat-screenshots-endpoint Oct 27, 2025
39 checks passed
@lohanidamodar lohanidamodar deleted the feat-screenshot-endpoint-stats branch October 28, 2025 00:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@shimonewman shimonewman shimonewman approved these changes

@Meldiron Meldiron Awaiting requested review from Meldiron

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@lohanidamodar @shimonewman