Skip to content

Add auth support#228

Merged
DoodleBobBuffPants merged 3 commits into
mainfrom
auth
Feb 9, 2026
Merged

Add auth support#228
DoodleBobBuffPants merged 3 commits into
mainfrom
auth

Conversation

@TheEadie
Copy link
Copy Markdown
Contributor

@TheEadie TheEadie commented Feb 9, 2026

Description

Allow users to authenticate with Flyway automatically by providing email, token, and agree-to-eula inputs. When both email and token are provided, the action runs flyway auth after installation.

Related issue

N/a

Check list

  • Mark if documentation changes are required.
  • Mark if tests were added or updated to cover the changes.

TheEadie and others added 2 commits February 9, 2026 14:55
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@DoodleBobBuffPants DoodleBobBuffPants left a comment

Choose a reason for hiding this comment

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

The change is fine, just some thoughts:

  • There are a few ways to authenticate, such as supplying permits / offline permits. Should we be adding support for those? What about documenting which environment variables to set vs adding inputs?
  • Should authenticating be a separate action, in particular for those users that would use offline permits and will have Flyway already available on self-hosted agents so they do not need to make external network calls

@TheEadie
Copy link
Copy Markdown
Contributor Author

TheEadie commented Feb 9, 2026

The change is fine, just some thoughts:

* There are a few ways to authenticate, such as supplying permits / offline permits. Should we be adding support for those? What about documenting which environment variables to set vs adding inputs?

* Should authenticating be a separate action, in particular for those users that would use offline permits and will have Flyway already available on self-hosted agents so they do not need to make external network calls

I suspect if you're using offline permits then you'll be in a non-internet connected environment and won't be able to make use of the Action. Similarly I think the seperate Action would hit the same issue. In those case we probably need better documentation on how to set up offline agents.

@DoodleBobBuffPants DoodleBobBuffPants merged commit 60f27d8 into main Feb 9, 2026
40 checks passed
@DoodleBobBuffPants DoodleBobBuffPants deleted the auth branch February 9, 2026 15:32
@TheEadie TheEadie changed the title Add authenication support Add auth support Feb 9, 2026
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.

2 participants