host verification disable now works

[joseluu]

PR summary

fixes #2592
This is a 3 lines change:

• GlobalCatalogV1 object now carries a disable_ssl_verification flag
• GlobalSearchV2 now initializes the BaseService parent with the proper value of disable_ssl_verification

Current vs new behavior

disabling host verification now works

Does this PR introduce a breaking change?

• Yes
• No

[diego-plan9]

Thanks @joseluu ! This would require a new version of platform-services-python-sdk that contains your fix to be released before working, right?

[diego-plan9]

Is this change intentional? While the verify flag in the signature of this function (and others) could use a revision for ensuring it is always a bool, I believe this line mostly coerces None into an actual boolean and should still be kept for preserving previous behavior.

[joseluu]

Thanks @joseluu ! This would require a new version of platform-services-python-sdk that contains your fix to be released before working, right?

Thanks to a suggestion from the platform-services-python-sdk maintainers, I just made a second commit avoiding the dependency on any platform-services-python-sdk modification.

[diego-plan9]

Thanks - unfortunately, a related PR was started in the meantime, #2607, which provides a more comprehensive fix (extending the ssl verification flag to calls to other services, and includes handling of the proxy arguments too). As it is a superset of this PR, I'm inclined to focus on landing #2607 if it gets into mergeable state before the next release.

[diego-plan9]

Thanks again - closing as #2607 was merged.