Bug #13547

[python sdk] does not respect ARVADOS_API_HOST_INSECURE when speaking to keep(-proxy) via SSL

Added by Ward Vandewege over 3 years ago. Updated about 3 years ago.

Status:
Resolved
Priority:
Normal
Assigned To:
Category:
-
Target version:
Start date:
07/02/2018
Due date:
% Done:

100%

Estimated time:
(Total: 0.00 h)
Story points:
-
Release:
Release relationship:
Auto

Subtasks

Task #13548: review 13547-respect-insecure-flag-when-talking-ssl-to-keepResolvedWard Vandewege

Associated revisions

Revision cdd57a73 (diff)
Added by Ward Vandewege over 3 years ago

13547: make the Python SDK respect the ARVADOS_API_HOST_INSECURE environment
variable when speaking with keep over SSL

refs #13547

Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <>

Revision 088268fc (diff)
Added by Ward Vandewege over 3 years ago

13547: if an instance of KeepClient has an api_client object with the insecure
flag set, and the ARVADOS_API_HOST_INSECURE environment variable is not set,
set the insecure flag on the KeepClient instance. Also set that flag on any
KeepService instances created by that KeepClient instance.

This makes arv-copy respect the ARVADOS_API_HOST_INSECURE value in the config
file(s) for the Arvados instances, when talking to keep proxy.

refs #13547

Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <>

Revision c1ab715a
Added by Ward Vandewege over 3 years ago

Merge branch '13547-respect-insecure-flag-when-talking-ssl-to-keep'

closes #13547

Arvados-DCO-1.1-Signed-off-by: Ward Vandewege <>

History

#1 Updated by Tom Morris over 3 years ago

  • Status changed from New to In Progress
  • Assigned To set to Ward Vandewege
  • Target version set to 2018-07-03 Sprint

#2 Updated by Tom Clegg over 3 years ago

Updated to prevent the insecure flag in the default config from leaking into other configs (e.g., arv-copy).

13547-respect-insecure-flag-when-talking-ssl-to-keep @ 76a2c8a1eb1771b5255e8582b7e9823d072e8ef4

https://ci.curoverse.com/job/developer-run-tests/779/

#3 Updated by Ward Vandewege over 3 years ago

  • Status changed from In Progress to Resolved
  • % Done changed from 0 to 100

#4 Updated by Tom Morris about 3 years ago

  • Release set to 13

Also available in: Atom PDF