FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

fix: update API token URL and Discover example (README + error message) by omri-rulf · Pull Request #44 · brightdata/sdk-python · GitHub

fix: update API token URL and Discover example (README + error message) - #44

Merged
shahar-brd merged 4 commits into
brightdata:mainfrom
omri-rulf:update_API_key_URL_to_current_CP_location
Jul 19, 2026
Merged

fix: update API token URL and Discover example (README + error message)#44
shahar-brd merged 4 commits into
brightdata:mainfrom
omri-rulf:update_API_key_URL_to_current_CP_location

Conversation

Copy link
Copy Markdown
Contributor

What

  • Replaced all references to https://brightdata.com/cp/api_keys (returns 404)
    with the current location: https://brightdata.com/cp/setting/users
    — including the token-missing error message users see on first run
  • Fixed the Discover example: result items have a link field, not url —
    the current example throws KeyError when copy-pasted (verified against live
    API response: keys are ['link', 'title', 'description', 'relevance_score'])

Why

Found during a hands-on SDK test session (Windows, Python 3.14, SDK v2.4.0).
Both issues break the first-run experience for new users.

A CP-side redirect for /cp/api_keys (covering already-shipped SDK versions)
is being handled separately.

shahar-brd merged commit a4a81a0 into brightdata:main Jul 19, 2026
3 checks passed
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
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.

2 participants


Back | FazBrowse Home | New Git URL