Re: [PATCH] Check that opts is not None before accessing.

[Ralph Bean]

On Fri, May 08, 2015 at 11:56:40AM -0400, Ralph Bean wrote:
> This should fix the periodic 500 error we get from koji-web in Fedora
> Infra's staging environment.
> Signed-off-by: Ralph Bean
>
> www/lib/kojiweb/util.py | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/www/lib/kojiweb/util.py b/www/lib/kojiweb/util.py
> index 89a5ddb..6610beb 100644
> --- a/www/lib/kojiweb/util.py
> +++ b/www/lib/kojiweb/util.py
> @@ -466,7 +466,7 @@ def taskScratchClass(task_object):
>      request = task_object['request']
>      if len(request) >= 3:
>          opts = request[2]
>    if opts.get('scratch'):
>    if opts and opts.get('scratch'):
>        return "scratch"
> return ""
>
> -- 
> 2.1.0

FYI, we miscommunicated about this in #fedora-releng yesterday and the
unpatched version got pushed out to prod.  This means that every now
and then, the koji frontpage fails with 500 (until the task with
`opts == None` scrolls off the "Recent Tasks" pane).