Summary

Allow hosting services to define URLs for the service.

Review Request #15141 — Created June 26, 2026 and submitted July 17, 2026, 10:19 a.m.

Information

Owner

david

Repository

Review Board

Branch

release-9.x

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

Hosting service subclasses have long been able to define URL patterns,

but these were always scoped under a repository ID. This change adds

another class attribute for defining URLs that are for the hosting

service generally.

Testing Done

Used this from within other changes.

Commits

Summary	ID
Allow hosting services to define URLs for the service. Hosting service subclasses have long been able to define URL patterns, but these were always scoped under a repository ID. This change adds another class attribute for defining URLs that are for the hosting service generally. Testing Done: Used this from within other changes.	lsqonktuunwspxoyxxznqkwtwuzlonks

Issues

Description	From	Last Updated
Order should be reversed here.	chipx86	July 7, 2026, 1:06 p.m.
This feels a bit noisy at this point. Possible cleanup: if patterns := self._hosting_service_url_patterns.pop(service_id, None): hosting_service_urls.remove_patterns(patterns) ...	chipx86	July 7, 2026, 1:56 p.m.

flake8 passed.

JSHint passed.

reviewboard/hostingsvcs/base/registry.py (Diff revision 1)
The issue has been resolved. Show all issues
```
Order should be reversed here.
```

reviewboard/hostingsvcs/base/registry.py (Diff revision 1)

The issue has been resolved. Show all issues

This feels a bit noisy at this point. Possible cleanup:

if patterns := self._hosting_service_url_patterns.pop(service_id,
                                                      None):
    hosting_service_urls.remove_patterns(patterns)

...

Commits:

	Summary	ID
	Allow hosting services to define URLs for the service. Hosting service subclasses have long been able to define URL patterns, but these were always scoped under a repository ID. This change adds another class attribute for defining URLs that are for the hosting service generally. Testing Done: Used this from within other changes.	lsqonktuunwspxoyxxznqkwtwuzlonks
	Allow hosting services to define URLs for the service. Hosting service subclasses have long been able to define URL patterns, but these were always scoped under a repository ID. This change adds another class attribute for defining URLs that are for the hosting service generally. Testing Done: Used this from within other changes.	lsqonktuunwspxoyxxznqkwtwuzlonks

Diff:

Revision 2 (+156 -58)

Show changes

	reviewboard/hostingsvcs/urls.py
	reviewboard/hostingsvcs/base/hosting_service.py
	reviewboard/hostingsvcs/base/registry.py

Checks run (2 succeeded)

flake8 passed.

JSHint passed.

Ship it!

Status:: Completed
Change Summary:: Pushed to release-9.x (efbfb6a)