Skip to content

Fix blocking of phone sessions when disabled on console #9447

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

Merged
merged 2 commits into from
Mar 5, 2025
Merged

Conversation

ChiragAgg5k
Copy link
Member

@ChiragAgg5k ChiragAgg5k commented Mar 5, 2025

What does this PR do?

Phone sessions where not blocked even made blocked via the console as the endpoint did not have auth group added to it.

Test Plan

image

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?

Copy link

github-actions bot commented Mar 5, 2025

Security Scan Results for PR

Docker Image Scan Results

🎉 No vulnerabilities found!

Source Code Scan Results

🎉 No vulnerabilities found!

Copy link

github-actions bot commented Mar 5, 2025

✨ Benchmark results

  • Requests per second: 1,048
  • Requests with 200 status code: 188,599
  • P99 latency: 0.176721621

⚡ Benchmark Comparison

Metric This PR Latest version
RPS 1,048 1,357
200 188,599 244,296
P99 0.176721621 0.13751713

@stnguyen90 stnguyen90 merged commit fab43a5 into 1.6.x Mar 5, 2025
29 checks passed
@ChiragAgg5k ChiragAgg5k deleted the pla-2607 branch March 12, 2025 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants