Skip to content

Conversation

@bnchrch
Copy link
Contributor

@bnchrch bnchrch commented Feb 4, 2025

What

Fixes #53158

How

Unhides client id and client secret and makes them required

@vercel
Copy link

vercel bot commented Feb 4, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback Feb 4, 2025 11:17pm
@octavia-squidington-iii octavia-squidington-iii added area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/instagram labels Feb 4, 2025
@bnchrch bnchrch marked this pull request as ready for review February 4, 2025 22:34
}
},
"required": ["access_token"]
"required": ["client_id", "client_secret", "access_token"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is the answer why optinal

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Ah sorry what I meant was in instagram 3.14 I could see client_id and client_secret under optional. But in the original report client_id and client_secret were missing from the optional section.

Was not sure why

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh nvm! Looks like it was an issue with an (unoticed) page refresh when I was developing!

@bnchrch bnchrch force-pushed the bnchrch/source-instagram/unhide-oauth-fields branch from b80b660 to e355629 Compare February 4, 2025 23:11
@bnchrch
Copy link
Contributor Author

bnchrch commented Feb 4, 2025

Currently trying to validate I can login with instagram / facebook and seem to be running into a scope issue
image

Looks like this is a valid reason to bring your own access token in this case

@bnchrch bnchrch merged commit 1e4dbcc into master Feb 5, 2025
31 checks passed
@bnchrch bnchrch deleted the bnchrch/source-instagram/unhide-oauth-fields branch February 5, 2025 22:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/instagram

4 participants