Merge pull request #2782 from ecordell/no-signing-whitelist
Revert "Only show signing UI when namespace is explicitly whitelisted"
This commit is contained in:
commit
4ec4b7c6e6
4 changed files with 3 additions and 11 deletions
|
@ -23,7 +23,7 @@ CLIENT_WHITELIST = ['SERVER_HOSTNAME', 'PREFERRED_URL_SCHEME', 'MIXPANEL_KEY',
|
|||
'CONTACT_INFO', 'AVATAR_KIND', 'LOCAL_OAUTH_HANDLER', 'DOCUMENTATION_LOCATION',
|
||||
'DOCUMENTATION_METADATA', 'SETUP_COMPLETE', 'DEBUG', 'MARKETO_MUNCHKIN_ID',
|
||||
'STATIC_SITE_BUCKET', 'RECAPTCHA_SITE_KEY', 'CHANNEL_COLORS',
|
||||
'TAG_EXPIRATION_OPTIONS', 'SIGNING_NAMESPACE_WHITELIST']
|
||||
'TAG_EXPIRATION_OPTIONS']
|
||||
|
||||
|
||||
def frontend_visible_config(config_dict):
|
||||
|
@ -269,9 +269,6 @@ class DefaultConfig(ImmutableConfig):
|
|||
# only private repositories can be returned.
|
||||
FEATURE_PUBLIC_CATALOG = False
|
||||
|
||||
# The namespaces which should have the ability to enable signing
|
||||
SIGNING_NAMESPACE_WHITELIST = ['coreos', 'quay']
|
||||
|
||||
# The namespace to use for library repositories.
|
||||
# Note: This must remain 'library' until Docker removes their hard-coded namespace for libraries.
|
||||
# See: https://github.com/docker/docker/blob/master/registry/session.go#L320
|
||||
|
|
|
@ -19,7 +19,7 @@
|
|||
</div>
|
||||
|
||||
<!-- Signing and Trust -->
|
||||
<div ng-if="repository.kind == 'image' && signingWhitelisted(repository)">
|
||||
<div ng-if="repository.kind == 'image'">
|
||||
<repository-signing-config repository="repository"></repository-signing-config>
|
||||
</div>
|
||||
|
||||
|
|
|
@ -90,10 +90,6 @@ angular.module('quay').directive('repoPanelSettings', function () {
|
|||
$scope.repository.is_public = newAccess == 'public';
|
||||
}, ApiService.errorDisplay('Could not change visibility'));
|
||||
};
|
||||
|
||||
$scope.signingWhitelisted = function(repository) {
|
||||
return Config.SIGNING_NAMESPACE_WHITELIST.indexOf(repository.namespace) !== -1;
|
||||
};
|
||||
}
|
||||
};
|
||||
return directiveDefinitionObject;
|
||||
|
|
|
@ -66,8 +66,7 @@ class TestConfig(DefaultConfig):
|
|||
SECURITY_SCANNER_API_TIMEOUT_SECONDS = 1
|
||||
|
||||
FEATURE_SIGNING = True
|
||||
SIGNING_NAMESPACE_WHITELIST = ['devtable']
|
||||
|
||||
|
||||
SIGNING_ENGINE = 'gpg2'
|
||||
|
||||
GPG2_PRIVATE_KEY_NAME = 'EEB32221'
|
||||
|
|
Reference in a new issue