Skip to content

Commit

Permalink
fix(deadline): Disable client SSL config for HTTP Render Queue (#167)
Browse files Browse the repository at this point in the history
Fixes #165
  • Loading branch information
horsmand authored Oct 13, 2020
1 parent b4bfd4e commit fe347fa
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 9 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
# Regex for counting the number of certificates in a cert chain
CERT_COUNT_RE = re.compile(r'-----BEGIN CERTIFICATE-----')

# Named tuples for storing arguments
# Named tuples for storing arguments
RenderQueue = namedtuple('RenderQueue','uri,scheme,address')
AwsSecret = namedtuple('AwsSecret','arn,region')
FileSecret = namedtuple('FileSecret', 'filepath')
Expand All @@ -42,7 +42,7 @@ def parse_args(args):
:param args: A list of command line arguments
:return: A configuration object containing the parsed arguments
"""

def _secret(value):
"""
A type function for converting args that represent secrets into a named Tuple
Expand All @@ -61,15 +61,15 @@ def _secret(value):
if match:
named_groups = match.groupdict()
return FileSecret( arn=named_groups['FilePath'] )

raise argparse.ArgumentTypeError('Given argument "%s" is not a valid secret' % value)

def _render_queue(value):
"""
A type function for converting args that represent render queue URI's into a named Tuple
:param value: The string representing the argument
:return: A RenderQueue named tuple
:return: A RenderQueue named tuple
:exception argparse.ArgumentTypeError: if the argument cannot be converted properly.
"""

Expand Down Expand Up @@ -130,15 +130,17 @@ def configure_deadline( config ):
:param config: The parsed configuration object
"""

repo_args = ['ChangeRepository','Proxy',config.render_queue.address]
if config.render_queue.scheme == 'http':
print( "Configuring Deadline to connect to the Render Queue (%s) using HTTP Traffic" % config.render_queue.address )
#Ensure SSL is disbaled
#Ensure SSL is disabled
call_deadline_command(['SetIniFileSetting','ProxyUseSSL','False'])
call_deadline_command(['SetIniFileSetting', 'ProxySSLCA', ''])
call_deadline_command(['SetIniFileSetting', 'ClientSSLAuthentication', 'NotRequired'])

else:
print("Configuring Deadline to connect to the Render Queue using HTTP Traffic")
print("Configuring Deadline to connect to the Render Queue using HTTPS Traffic")
call_deadline_command(['SetIniFileSetting','ProxyUseSSL','True'])

try:
Expand Down
4 changes: 2 additions & 2 deletions packages/aws-rfdk/lib/deadline/test/asset-constants.ts
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,6 @@ export const REPO_DC_ASSET = {
};

export const RQ_CONNECTION_ASSET = {
Bucket: 'AssetParameters63694479464606109bdbd3525fb2bef7b2abfbf196d8a132832c8e5d8a3c4796S3BucketF3231D14',
Key: 'AssetParameters63694479464606109bdbd3525fb2bef7b2abfbf196d8a132832c8e5d8a3c4796S3VersionKeyE501DFB8',
Bucket: 'AssetParameters89a29e05a2a88ec4d4a02e847e3c3c9461d0154b326492f4cad655d4ca0bda98S3BucketC22E185C',
Key: 'AssetParameters89a29e05a2a88ec4d4a02e847e3c3c9461d0154b326492f4cad655d4ca0bda98S3VersionKey0833D670',
};

0 comments on commit fe347fa

Please sign in to comment.