Skip to content

[DOP-24163] add more verbose error message#191

Merged
TiGrib merged 5 commits intodevelopfrom
feature/DOP-24163
Mar 19, 2025
Merged

[DOP-24163] add more verbose error message#191
TiGrib merged 5 commits intodevelopfrom
feature/DOP-24163

Conversation

@TiGrib
Copy link
Contributor

@TiGrib TiGrib commented Mar 19, 2025

Change Summary

More verbose error message from keycloak

Related issue number

[DOP-24163]

Checklist

  • Commit message and PR title is comprehensive
  • Keep the change as small as possible
  • Unit and integration tests for the changes exist
  • Tests pass on CI and coverage does not decrease
  • Documentation reflects the changes where applicable
  • docs/changelog/next_release/<pull request or issue id>.<change type>.rst file added describing change
    (see CONTRIBUTING.rst for details.)
  • My PR is ready to review.

@TiGrib TiGrib requested a review from dolfinus March 19, 2025 12:21
@TiGrib TiGrib self-assigned this Mar 19, 2025
@codecov
Copy link

codecov bot commented Mar 19, 2025

Codecov Report

Attention: Patch coverage is 0% with 1 line in your changes missing coverage. Please review.

Project coverage is 91.88%. Comparing base (a30544f) to head (fa82d9b).
Report is 110 commits behind head on develop.

Files with missing lines Patch % Lines
...rentgen/server/providers/auth/keycloak_provider.py 0.00% 1 Missing ⚠️
Additional details and impacted files
@@             Coverage Diff             @@
##           develop     #191      +/-   ##
===========================================
- Coverage    91.90%   91.88%   -0.02%     
===========================================
  Files          197      197              
  Lines         4892     4893       +1     
  Branches       342      342              
===========================================
  Hits          4496     4496              
- Misses         327      328       +1     
  Partials        69       69              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@dolfinus dolfinus added the ci:skip-changelog Add this label to skip changelog file check label Mar 19, 2025
@TiGrib TiGrib enabled auto-merge (squash) March 19, 2025 13:09
@TiGrib TiGrib disabled auto-merge March 19, 2025 13:12
@TiGrib TiGrib merged commit 1480a62 into develop Mar 19, 2025
13 of 15 checks passed
@TiGrib TiGrib deleted the feature/DOP-24163 branch March 19, 2025 13:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci:skip-changelog Add this label to skip changelog file check

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants