Add browser based authentication using Taboola's SSO#1
Closed
henryabra wants to merge 11 commits intotaboola:mainfrom
Closed
Add browser based authentication using Taboola's SSO#1henryabra wants to merge 11 commits intotaboola:mainfrom
henryabra wants to merge 11 commits intotaboola:mainfrom
Conversation
Fixed server cleanup on failure flow
Added system termination signal listening to clean up callback server (if running)
…_token` based on client credentials configuration. - Fixed state extraction logic in OAuth callback to handle both buggy and correct formats. - Fixed handler support for `clear_auth_token` in `realize_server.py`. - Fixed campaign handlers to use correct API endpoint paths (`/items` instead of `/campaign_items`).
… based on client credentials. - Added `tools/assets` to package data for proper asset management. - Enhanced documentation to reflect browser-based authentication flow and updated installation details.
…or better modularity. - Updated registry to reference the new `auth_handlers` namespace. - Refactored tests to support both credential and browser-based authentication tools. - Added `tomli` dependency for Python versions below 3.11.
…e, header generation, and error handling. - Introduced tests for `browser_authenticate` and `clear_auth_token` handlers. - Verified browser-based authentication integration with system and tool availability logic.
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
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Browser Authentication Flow
Summary
Added browser-based authentication support to allow users to authenticate without API credentials, making the MCP server more accessible.
Changes
get_auth_tokentool is now conditionally shown only when client credentials are configuredTechnical Details
browser_authenticateandclear_auth_tokentools for managing browser-based authenticationRealizeAuthwhen credentials exist, falls back toBrowserAuthotherwiseThis change makes the Realize MCP server more user-friendly by removing the requirement for API credentials upfront, while maintaining support for traditional credential-based authentication for automated workflows.