openwisp / openwisp-radius

99%

LAST BUILD ON BRANCH master

branch: master

CHANGE BRANCH

x

Reset

• master
• 1.0
• 1.1
• Issues/550-failure-on-importing-utf-16-files
• MaxQuotaReached-override-configured-replies
• Selenium-tests-for-Dependabot-PRs-537
• allow-empty-octets
• chore/port-changelog-1.1.2
• cleanup_stale_radacct_hours
• dependabot/github_actions/actions/checkout-5
• dependabot/github_actions/pypa/gh-action-pypi-publish-1.12.3
• dependabot/github_actions/pypa/gh-action-pypi-publish-1.12.4
• dependabot/pip/cssselect-approx-eq-1.3.0
• dependabot/pip/dj-rest-auth-gte-6.0-and-lt-7.1
• dependabot/pip/django-ipware-gte-5.0-and-lt-7.1
• dependabot/pip/django-redis-approx-eq-6.0.0
• dependabot/pip/djangosaml2-1.10.1
• dependabot/pip/djangosaml2-1.11.1
• dependabot/pip/freezegun-approx-eq-1.5.2
• dependabot/pip/freezegun-approx-eq-1.5.4
• dependabot/pip/freezegun-approx-eq-1.5.5
• dependabot/pip/lxml-approx-eq-5.3.1
• dependabot/pip/lxml-approx-eq-5.4.0
• dependabot/pip/lxml-approx-eq-6.0.0
• dependabot/pip/lxml-approx-eq-6.0.1
• dependabot/pip/pydyf-gte-0.10-and-lt-0.12
• dependabot/pip/weasyprint-gte-59-and-lt-66
• dependabot/pip/weasyprint-gte-65-and-lt-67
• feat/batch-creation-async
• fix-accounting-on
• fix-ci-djangosaml2
• fix-monitoring-integration
• fix-perform_change_of_authorization
• fix/467-postauthview-validation
• fix/crash-radiusgroup-inlines-creation
• format-by-prettier
• improve-db
• issues/510-fix-sms-message-translation
• issues/551
• issues/581-django-5
• issues/611-fix-timezone-handling-tests
• issues/615-simultaneous-use
• issues/617-handle-stale-sessions-accounting-on
• issues/634-counters-return-multiple-values
• issues/643-coa
• qa
• refactor-monthly_subscription
• release-1.1.1
• release/1.1.2
• replicate-version-branch
• switch-to-prettier
• tidy-up
• weasyprint

Build # 18102722890

Build Type

push

github

Committed by web-flow

Commit Message

[change] Change of Authorization: added replies, handled MaxQuotaReached, refactored #643

- When the RADIUS group of a user is changed, the previous implementation
  did not include the RADIUS replies of the new group in the CoA packet,
  which could result in incomplete or incorrect authorization updates.
  This has been fixed by including the RADIUS replies of the new group.
- Added handling of ``MaxQuotaReached`` exceptions: when the RADIUS group
  of a user is changed but their limits are already exceeded, instead of
  sending a CoA Request (to modify a session), we now send a Disconnect
  Message to log out the user.
- Refactored code to reuse counter and attribute retrieval from
  ``AuthorizeView._check_counters``, reducing duplication and improving
  maintainability.

Closes #643

Run Details

3989 of 4041 relevant lines covered (98.71%)

11.84 hits per line