Skip to content

Conversation

@elsheikhams99
Copy link
Collaborator

@elsheikhams99 elsheikhams99 commented Jul 12, 2025

Key Fixes & Enhancements:

  1. Bug Resolution & Testing:

    • Thoroughly tested authentication, data flow, and action execution for all connectors.
    • Covered edge cases including incorrect credentials and unsupported schemas.
  2. Authentication Method Support:

    • Added missing authentication methods:

      • AuthenticationSchema.BASIC
      • AuthenticationSchema.API_KEY
    • All supported schemas now:

      [
        AuthenticationSchema.BEARER_TOKEN,
        AuthenticationSchema.OAUTH1,
        AuthenticationSchema.OAUTH2,
        AuthenticationSchema.API_KEY,
        AuthenticationSchema.BASIC,
        AuthenticationSchema.NO_AUTH
      ]
  3. Dynamic Schema Retrieval & Validation:

    • Re-implemented integration.py to allow the SDK to:

      • Dynamically fetch the supported authentication methods per integration.
      • Raise precise errors when the wrong schema or missing parameters are used.
  4. Error Handling Improvements:

    • Invalid schema:

      raise ValueError(f"Authentication schema {authentication_schema.value} is not supported for this integration. Supported authentication methods: {self.authentication_methods}")
    • Missing required parameters:

      raise ValueError(f"Parameters {required_params_names} are required for {self.name} {authentication_schema.value} authentication. Please provide the parameters in the data dictionary.")
  5. Known Limitation (Composio SDK):

    • If no data is passed (or it's incorrect), and the initiation call is made, the SDK does not raise an error at connection time. The failure becomes apparent only when using the tool/agent.

@elsheikhams99 elsheikhams99 requested a review from hadi-aix July 12, 2025 13:23
@hadi-aix hadi-aix merged commit 7e59519 into development Jul 13, 2025
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.

3 participants