koji

#1745 allow comma delimiter for allowed_methods
Merged by mikem. Opened by tkopecek.
tkopecek/koji issue1744  into  master

Download 1745.patch

Example config says, that comma is allowed, but it was not true. I've made it more in line with rest of configs (allowing only comma)

Fixes: https://pagure.io/koji/issue/1744

Wouldn't the first split return a list, causing the second split to return AttributeError?

rebased onto a76f02b190b91fb21cecf7872ca93f646b416cf0

:thumbsup:

Is there a chance that someone is relying on splitting by whitespace?

There is a slight chance, but I don't know about any other deployment using this plugin than ours.

ok then :smile: :thumbsup:

Commit 47017397 fixes this pull-request

Pull-Request has been merged by mikem

Metadata Update from @jcupova:
- Pull-request tagged with: testing-done
Metadata
None

testing-done

Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.