airbytehq / airbyte

The leading data integration platform for ETL / ELT data pipelines from APIs, databases & files to data warehouses, data lakes & data lakehouses. Both self-hosted and Cloud-hosted.
https://airbyte.com
Other
15.41k stars 3.98k forks source link

[connector-builder] Error when passing token as a path parameter to ADS API #37314

Open deepapt opened 4 months ago

deepapt commented 4 months ago

Connector Name

Source - Builder

Connector Version

0

What step the error happened?

Configuring a new connector

Relevant information

I am trying to support the following APIs in Airbyte Builder in the cloud. I'm encountering an error when trying to pass a token as a path parameter to the ADS API in Airbyte Builder. Here are the details:

Steps to Reproduce:

ValueError: Jinja macro has undeclared variables: {'token'}. Context: {context}

{ "end": 1712230022, "token": "82fd7b1a22b98f2ae46f6ede85382a2b" }

Relevant log output

No response

Contribute

marcosmarxm commented 4 months ago

Hello @deepapt I added the issue to the connector builder team backlog.

jspooner commented 4 months ago

Passing the token in the URL is not secure and I would not recommend Airbyte support this.

@deepapt our API partner is currently removing this token from the URL and supporting header auth.