Review Board 2.0 RC 3 Release Notes¶

Experimental Web Hooks¶

• Added experimental “close-submitted” Web Hook support for Bitbucket, Beanstalk, and Google Code.

  Bitbucket, Beanstalk, GitHub, and Google Code repositories can now be configured to close review requests referenced in a commit message (or identified by a commit ID) by using Web Hooks.

  To use this, configure a WebHook and provide the URL for the hook.

  For Bitbucket: http://<server>/repos/<repo_id>/hooks/bitbucket/close-submitted/

  For Beanstalk: http://<server>/repos/<repo_id>/hooks/beanstalk/close-submitted/

  For GitHub: http://<server>/repos/<repo_id>/hooks/github/close-submitted/

  For Google Code: http://<server>/repos/<repo_id>/hooks/googlecode/close-submitted/

  Note

  The 2.0 RC 2 release notes cover the usage of these hooks in more detail, though in RC 2, these webhooks were named “post-receive” instead of “close-submitted”.

  Documentation on using these will be available at a later date. For now, the feature is experimental.

  Patches by Bhushan Ramnani.

• “close-submitted” Web Hooks can now match by commit ID.

  The post-review Web Hooks for GitHub, Bitbucket, Beanstalk, and Google Code, which close review requests when commits are pushed, can now look up the proper review request through the Commit ID field, instead of requiring a URL in the commit message. This requires that the review request’s commit ID field matches the resulting SHA1 of the commit.

  Patch by Anselina Chia.

Other¶

• Improved the look of generated e-mails.

Better Upgrades¶

Installing a new version of an extension will cause Review Board to re-install all bundled media files.

This removes the need for administrators to disable and re-enable an extension in order to get things working again.

Media Packaging¶

• Source media files (JavaScript, CSS, and LessCSS) can now be removed from built extension packages.

  This behavior can be triggered by adding the following to the package’s setup.cfg:

  [build_static_files]
  remove_source_files = 1
  

JavaScript¶

• Improved support in RB.ReviewGroup and RB.Repository.

  These resource models, used to communicate with the API, have been improved to better handle parsing payloads returned from the API.

• Added RB.DefaultReviewer for parsing payloads for the Default Reviewer API.

  This is not feature-complete, but can be used to fetch basic details on a Default Reviewer entry.

Sandboxing¶

• Added sandboxing for several extension hook points.

  Many of the hook points Review Board offers now protect the rest of the application from crashing if an extension raises an exception. Currently, ReviewRequestApprovalHook, NavigationBarHook, CommentDetailDisplayHook, TemplateHook, and the Action Hooks are sandboxed.

  Patches by Edwin Zhang.

New Signals¶

• Added review_request_publishing, review_publishing, and reply_publishing signals.

  These signals are emitted before the publishing process starts on review requests, reviews, and replies. Extensions can connect to these to perform pre-publish validation. They can raise a PublishError to abort the publish.

  The error message provided to PublishError will be reflected in the API’s error message.

  Patch by Mark Côté.

Authentication Backends¶

• Custom AuthBackends can now raise a UserQueryError when looking up errors.

  These errors will be represented in the web API though 226 - User Query Error.

General¶

• Fixed a regression where Gravatars wouldn’t display until Gravatar support was toggled off and back on.

• Fixed a regression when resetting user passwords. (Bug #3301)

• Fixed various browser compatibility issues with Firefox.

  Patch by Matthew N.

• Duplicate Markdown Reference links are no longer shown in replies to comments.

• Diffs containing leading slashes are normalized to remove leading slashes, fixing lookups.

Review Requests¶

• Fixed displaying and handling errors when publishing a review request fails.

  Patch by Mark Côté.

• Review requests created through the New Review Request page will once again show a draft banner.

• Fixed escaping of content in the bugs field. (Bug #3244)

Dashboard and DataGrids¶

• The Hide closed link on the All Review Requests page no longer impacts the dashboard. (Bug #3319)

• Sorting on non-existing columns in the Dashboard no longer breaks.

Administrative¶

• Upgrades from pre-1.7 databases now work again.

• Condensing diffs in the database should no longer use up all memory.

• Fixed an issue parsing some payloads on GitHub in the post-receive Web Hook.

• Fixed Unicode errors with LDAP authentication when the user dn contains non-ASCII characters.

  Patch by Georgy Dyuldin.

• Fixed breakages in the administration UI when viewing review requests with unicode characters in the summary. (Bug #3325)

Repository Support¶

• Fixed authentication with Subversion backends. (Bug #3303)

• Fixed a “Need more than 1 value to unpack” error with ClearCase. (Bug #3305)

  Patch by David Bodor.

• Fixed permission lookups when creating repositories through the web API. (Bug #3307)

Extensions¶

• Fixed leaking of variables between hook-provided templates.

Web API¶

• Unicode values in GET query strings no longer causes HTTP 500 errors.