Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
1 task
deepapt opened this issue Apr 14, 2024 · 2 comments
Open
1 task

Comments

@deepapt
Copy link

deepapt commented Apr 14, 2024

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

  • Yes, I want to contribute
@deepapt deepapt added area/connectors Connector related issues needs-triage type/bug Something isn't working labels Apr 14, 2024
@marcosmarxm marcosmarxm changed the title Error when passing token as a path parameter to ADS API [connector-builder] Error when passing token as a path parameter to ADS API Apr 15, 2024
@marcosmarxm
Copy link
Member

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

@jspooner
Copy link

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.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants