Skip to content

host verification disable now works #2593

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
joseluu wants to merge 2 commits into from
+2 −1
Closed

host verification disable now works #2593

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
2 commits
Select commit Hold shift + click to select a range
1fcee1e
fix host verification disable not working https://github.com/Qiskit/q…
joseluu Feb 6, 2026
48ea275
fix disabling ssl verification not requiring a new version of platfo…
joseluu Feb 27, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
5 changes: 3 additions & 2 deletions qiskit_ibm_runtime/accounts/account.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -314,9 +314,10 @@ def resolve_crn(self) -> None:
def list_instances(self) -> list[dict[str, Any]]:
"""Retrieve all crns with the IBM Cloud Global Search API."""
iam_url = get_iam_api_url(self.url)
authenticator = IAMAuthenticator(self.token, url=iam_url)
authenticator = IAMAuthenticator(self.token, url=iam_url, disable_ssl_verification=(not self.verify))
client = GlobalSearchV2(authenticator=authenticator)
catalog = GlobalCatalogV1(authenticator=authenticator)
catalog = GlobalCatalogV1(authenticator=authenticator,
disable_ssl_verification=(not self.verify))
client.set_service_url(get_global_search_api_url(self.url))
catalog.set_service_url(get_global_catalog_api_url(self.url))
search_cursor = None
Expand Down
2 changes: 1 addition & 1 deletion qiskit_ibm_runtime/qiskit_runtime_service.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -378,7 +378,7 @@ def _discover_account(
) -> Account:
"""Discover account for ibm_cloud and ibm_quantum_platform channels."""
account = None
verify_ = verify or True
verify_ = verify
Copy link
Copy Markdown
Member

@diego-plan9 diego-plan9 Feb 10, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

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.

if name:
if filename:
if any([channel, token, url]):
Expand Down
Loading