• 
      

    Use registry get_defaults() to return built-in authentication backends.

    Review Request #12213 — Created March 26, 2022 and submitted — Latest diff uploaded

    Information

    Review Board
    release-5.0.x

    Reviewers

    Entry points are fantastic, and they're a great way for us to allow
    third parties to integrate their own plugins with Review Board.
    Unfortunately, if there's a VersionConflict present in the install, no
    entry points will load, even if things will otherwise work. This means
    that no alternative authentication backends will load, which is extra
    bad because this may make it particulary difficult for administrators to
    log in to even start debugging the issue (such as looking at the log
    viewer in the admin).

    This change moves the built-in authentication backends into the
    registry's get_defaults() method, rather than using entry points. Any
    third-party services will still be loaded from entry points afterwards.

    Started with a fresh database and virtualenv. Saw that built-in
    authentication backends were available in the authentication page of the
    admin site.

    Commits

    Files