Skip to content

client(dataall_url=...): connect from the data.all front page, 0.6.0 - #56

Merged
TejasRGitHub merged 2 commits into
data-dot-all:mainfrom
TejasRGitHub:feat/client-from-frontend
Sep 25, 2026
Merged

TejasRGitHub merged 2 commits into
data-dot-all:mainfrom
TejasRGitHub:feat/client-from-frontend

Conversation

@TejasRGitHub

@TejasRGitHub TejasRGitHub commented Sep 25, 2026 •

Copy link
Copy Markdown
Contributor

Feature or Bugfix

  • Feature

Detail

Connect with nothing but the data.all front page URL:

import dataall_sdk as dataall
client = dataall.client(dataall_url="https://dataall.example.com")
client.list_organizations()
  • No code change in dataall_sdk.client(): it already forwards keyword arguments to DataallClient.client(), which gains dataall_url in Connect from a data.all front page URL (client(dataall_url=...)), 0.6.0 dataall-core#55. The discovered profile is saved under the page's host name (or profile=) so later calls skip the lookup; the first API call runs the OidcBrowserAuth browser login for OIDC deployments, or the Cognito flow for Cognito ones.
  • Tutorial and README updated: OidcBrowserAuth, the new profile fields (scopes, fallback_redirect_uri, frontend_url) and the front page section.
  • Functional tests for client(dataall_url=...) (profile discovered and saved, reused on the second call, missing values raise MissingParametersException).
  • Version 0.4.3 -> 0.6.0 and __version__ fixed (it still said 0.3.0a1). Requires dataall-core>=0.6.0,<0.7.0.
  • poetry.lock regenerated against the released dataall-core 0.6.0; verified in a fresh venv installing from PyPI.

Relates

Security

  • Input: dataall_url is fetched by dataall-core with two httpx GETs and parsed with anchored regular expressions; the values only populate profile fields the user could type by hand. No eval, no deserialization beyond response.text.
  • AuthN/AuthZ: unchanged; tokens are obtained and validated exactly as for a hand-written profile.
  • Cryptography: none added.
  • No new policies, roles or users.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

🤖 Generated with Claude Code

The SDK passes dataall_url through to DataallClient.client, which now
discovers and saves the profile from the data.all front page. Version
0.6.0 (also fixes __version__, which still said 0.3.0a1).

Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@TejasRGitHub
TejasRGitHub marked this pull request as ready for review September 25, 2026 16:58
Co-Authored-By: Claude Fable 5.1 <noreply@anthropic.com>
@TejasRGitHub
TejasRGitHub merged commit 886255d into data-dot-all:main Sep 25, 2026
2 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant