[PATCH] views: Don't render token section of user profile if REST API disabled

Stephen Finucane stephen at that.guru
Thu Jan 4 22:30:58 AEDT 2018


On Tue, 2017-12-19 at 16:41 +1100, Andrew Donnellan wrote:
> In profile.html, if settings.ENABLE_REST_API == False, trying to
> render a
> link to the generate_token page will raise a NoReverseMatch
> exception, so
> we shouldn't render that. In any case, if the REST API is disabled,
> we
> really shouldn't render the API token section of the page at all.
> 
> Only render the API token and generation link if
> settings.ENABLE_REST_API
> is True.
> 
> Cc: stable
> Reported-by: Tomas Novotny <tomas at novotny.cz>
> Closes: #138 ("NoReverseMatch exception on user login with disabled
> REST API")
> Fixes: 85c8f369204a ("views: Provide a way to view, (re)generate
> tokens")
> Signed-off-by: Andrew Donnellan <andrew.donnellan at au1.ibm.com>

Looks good to me. Merged and backported to 'stable/2.0'.

Reviewed-by: Stephen Finucane <stephen at that.guru>



More information about the Patchwork mailing list