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

Feature flag to disallow Meltano calling Hub outside of add and discover #7095

Closed
edgarrmondragon opened this issue Dec 15, 2022 · 2 comments · Fixed by #7097
Closed

Feature flag to disallow Meltano calling Hub outside of add and discover #7095

edgarrmondragon opened this issue Dec 15, 2022 · 2 comments · Fixed by #7097
Assignees

Comments

@edgarrmondragon
Copy link
Collaborator

edgarrmondragon commented Dec 15, 2022

Meltano Version

2.11.1

Python Version

NA

Bug scope

CLI (options, error messages, logging, etc.)

Operating System

NA

Description

With the recent switch to API Gateway and Netlify redirects, users that had whitelisted the now legacy Hub API (https://hub.meltano.com/meltano/api/v1/) are seeing failures because of the redirect to the Gateway endpoint.

See https://meltano.slack.com/archives/C01TCRBBJD7/p1671112207278089.


Back when I implemented meltano add from Hub, I decided to allow calls outside of add and discover, so users not yet onboarded to lockfiles would still be able to use their plugins without having to add those to a custom discovery.yml.

This can be disabled (as a breaking change) in our next major release (3.0).

Code

No response

@edgarrmondragon edgarrmondragon self-assigned this Dec 15, 2022
@WillDaSilva
Copy link
Member

Relates to:

@aaronsteers
Copy link
Contributor

aaronsteers commented Feb 9, 2023

Back when I implemented meltano add from Hub, I decided to allow calls outside of add and discover, so users not yet onboarded to lockfiles would still be able to use their plugins without having to add those to a custom discovery.yml.

At this point, we should probably disable that behavior and have Meltano fail when a plugin is not custom and doesn't have a lockfile.

@edgarrmondragon - I think we have to hold this until 3.0, if it would break existing users (and there are a lot) who have not yet locked their plugin files. Can you confirm?

Related: I think there was a decision at some point to allow plugins to get their definitions from the hub (if not locked) vs getting from discovery.yml. But reverting to a check against the bundled or web-served discovery.yml would also be a breaking change at this point, since there's likely been significant drift and bug fixes on the Hub that users have already adapted to.

@aaronsteers aaronsteers changed the title bug: Meltano calls Hub during stages other than add and discover Future (3.0) Change: Disallow Meltano calling Hub outside of add and discover Feb 9, 2023
@aaronsteers aaronsteers added this to the Meltano 3.0 milestone Feb 9, 2023
@aaronsteers aaronsteers removed the kind/Bug Something isn't working label Feb 9, 2023
@WillDaSilva WillDaSilva changed the title Future (3.0) Change: Disallow Meltano calling Hub outside of add and discover Disallow Meltano calling Hub outside of add and discover Apr 12, 2023
@WillDaSilva WillDaSilva changed the title Disallow Meltano calling Hub outside of add and discover Feature flag to disallow Meltano calling Hub outside of add and discover Apr 12, 2023
@WillDaSilva WillDaSilva removed this from the Meltano 3.0 milestone Apr 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
3 participants