dev/core#2141 - Tighten up new page `civicrm/oauth-client/return`
authorTim Otten <totten@civicrm.org>
Tue, 10 Nov 2020 22:25:21 +0000 (14:25 -0800)
committerTim Otten <totten@civicrm.org>
Tue, 10 Nov 2020 22:49:21 +0000 (14:49 -0800)
commit7121e3c1543f5f91c08d7a5170c2ccace9c59d98
tree0b4a672acf11039f9c758c3ae82f461155843efd
parent4fdca40a4ced5d81a8d36785d2f7b6d3f57a7b98
dev/core#2141 - Tighten up new page `civicrm/oauth-client/return`

Overview
--------

The route `civicrm/oauth-client/return` is added in 5.32 as the main "Redirect URL".

In normal usage, the page shouldn't be visible to a user (because the
developer should define some alternative UI) -- but one might see it (a)
during development, (b) if there's an error, or (c) if a clever user mucks
about.

Improvements:

* Error handling
    * Present error messages more nicely
    * Record errors in the log
    * Report more info via hook_oauthReturnError
* Other UI
    * Redact token details (dependent upon permission `manage OAuth client secrets`)
    * Set a more sensibile page title
    * Make output blobs conditional and collapsible
ext/oauth-client/CRM/OAuth/Page/Return.php
ext/oauth-client/templates/CRM/OAuth/Page/Return.tpl