Skip to content
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

Exception strings #12975

Open
wants to merge 2 commits into
base: main
Choose a base branch
from
Open

Exception strings #12975

wants to merge 2 commits into from

Conversation

nijel
Copy link
Member

@nijel nijel commented Nov 8, 2024

Proposed changes

Checklist

  • Lint and unit tests pass locally with my changes.
  • I have added tests that prove my fix is effective or that my feature works.
  • I have added documentation to describe my feature.
  • I have squashed my commits into logic units.
  • I have described the changes in the commit messages.

Other information

@nijel nijel added this to the 5.9 milestone Nov 8, 2024
@nijel nijel enabled auto-merge (rebase) November 8, 2024 18:43
@@ -1389,7 +1408,8 @@
try:
parse_query(query_string)
except Exception as error:
raise ValidationError(f"Could not parse query string: {error}") from error
msg = f"Could not parse query string: {error}"
raise ValidationError(msg) from error

Check warning

Code scanning / CodeQL

Information exposure through an exception Medium

Stack trace information
flows to this location and may be exposed to an external user.
@@ -1505,7 +1525,8 @@
try:
parse_query(query_string)
except Exception as error:
raise ValidationError(f"Could not parse query string: {error}") from error
msg = f"Could not parse query string: {error}"
raise ValidationError(msg) from error

Check warning

Code scanning / CodeQL

Information exposure through an exception Medium

Stack trace information
flows to this location and may be exposed to an external user.
Copy link

codecov bot commented Nov 8, 2024

Codecov Report

Attention: Patch coverage is 40.36036% with 331 lines in your changes missing coverage. Please review.

Project coverage is 90.87%. Comparing base (30e2a38) to head (216b5b5).
Report is 8 commits behind head on main.

Files with missing lines Patch % Lines
weblate/api/views.py 38.46% 32 Missing ⚠️
weblate/utils/views.py 21.42% 22 Missing ⚠️
weblate/utils/environment.py 25.00% 12 Missing ⚠️
weblate/auth/models.py 28.57% 10 Missing ⚠️
weblate/checks/parser.py 16.66% 10 Missing ⚠️
weblate/addons/views.py 33.33% 8 Missing ⚠️
weblate/formats/ttkit.py 0.00% 8 Missing ⚠️
weblate/trans/backups.py 33.33% 8 Missing ⚠️
weblate/trans/management/commands/import_json.py 42.85% 8 Missing ⚠️
weblate/trans/views/hooks.py 0.00% 8 Missing ⚠️
... and 68 more
Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main   #12975      +/-   ##
==========================================
- Coverage   91.09%   90.87%   -0.23%     
==========================================
  Files         598      598              
  Lines       61450    61684     +234     
  Branches     6373     6373              
==========================================
+ Hits        55980    56054      +74     
- Misses       3805     3965     +160     
  Partials     1665     1665              
Files with missing lines Coverage Δ
weblate/accounts/forms.py 89.16% <100.00%> (+0.01%) ⬆️
...eblate/addons/management/commands/install_addon.py 96.22% <100.00%> (+0.22%) ⬆️
weblate/auth/management/commands/createadmin.py 89.79% <100.00%> (+0.43%) ⬆️
weblate/billing/models.py 90.19% <100.00%> (+0.05%) ⬆️
weblate/billing/views.py 75.32% <100.00%> (+0.65%) ⬆️
weblate/formats/tests/test_convert.py 98.80% <100.00%> (-0.01%) ⬇️
weblate/formats/tests/test_formats.py 99.78% <100.00%> (+<0.01%) ⬆️
weblate/lang/views.py 89.42% <100.00%> (+0.10%) ⬆️
weblate/machinery/baidu.py 100.00% <100.00%> (ø)
weblate/machinery/tests.py 99.67% <100.00%> (-0.01%) ⬇️
... and 85 more

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.

1 participant