diff options
author | David Lawrence <dkl@mozilla.com> | 2017-06-01 01:09:52 +0200 |
---|---|---|
committer | David Lawrence <dkl@mozilla.com> | 2017-06-01 01:09:52 +0200 |
commit | 3a8db63716e1846323f4138c1628ef1a17eabb42 (patch) | |
tree | 1a7b698c15f00760dba6e3e8ed0d70337684828b /extensions/PhabBugz/lib | |
parent | 77658b1060a8761c15dfbedefce52eb1721aacc3 (diff) | |
download | bugzilla-3a8db63716e1846323f4138c1628ef1a17eabb42.tar.gz bugzilla-3a8db63716e1846323f4138c1628ef1a17eabb42.tar.xz |
Bug 1368066 - When using bmo auth delegation from Phabricator, the confirmation should be skipped similar to mozreview
Diffstat (limited to 'extensions/PhabBugz/lib')
-rw-r--r-- | extensions/PhabBugz/lib/Config.pm | 20 |
1 files changed, 20 insertions, 0 deletions
diff --git a/extensions/PhabBugz/lib/Config.pm b/extensions/PhabBugz/lib/Config.pm index 686198a25..a0b7ef2e5 100644 --- a/extensions/PhabBugz/lib/Config.pm +++ b/extensions/PhabBugz/lib/Config.pm @@ -35,6 +35,26 @@ sub get_param_list { type => 't', default => '', }, + { + name => 'phabricator_auth_callback_url', + type => 't', + default => '', + checker => sub { + my ($url) = (@_); + return 'must be an HTTP/HTTPS absolute URL' unless $url =~ m{^https?://}; + return ''; + } + }, + { + name => 'phabricator_app_id', + type => 't', + default => '', + checker => sub { + my ($app_id) = (@_); + return 'must be a hex number' unless $app_id =~ /^[[:xdigit:]]+$/; + return ''; + } + } ); return @params; |