Populate hosting services on initialization.

Review Request #6319 — Created Sept. 12, 2014 and submitted

Information

Review Board
release-2.0.x
e728829...

Reviewers

Hosting services were only populated when something first made use of
them. This meant that, until that happened, any hosting service URLs
weren't actually registered, and would return a 404.

We now populate when initializing. This incurs a small startup penalty,
but one that would be soon hit anyway, and is necessary for hosting
service URLs to be reliable.

This has been broken since hosting service URLs were introduced in 2.0.

Tested with an HTTP request to a URL as the first-processed request. Before
this change, that request would result in a generic 404. After the change,
I got a suitable error from the hook.

reviewbot
  1. Tool: Pyflakes
    Processed Files:
        reviewboard/hostingsvcs/service.py
    
    
    
    Tool: PEP8 Style Checker
    Processed Files:
        reviewboard/hostingsvcs/service.py
    
    
  2. 
      
david
  1. Ship It!

  2. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-2.0.x (433ac36)
Loading...