-
Notifications
You must be signed in to change notification settings - Fork 1.8k
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
Support non-classpath:
protocols in Spring
#2524
Conversation
… reallyinsane-1540
The changes make sense to me. I don't have a spring environment set up for testing it with different protocols, but we rely on spring's resource locator this should at least be a step in the right direction even if there are some protocols that have some unfound issues still. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
-
Fix is specific to the Spring Resource Accessor.
- Existing code tidied up and file protocol added.
- New integration test added to validate the file protocol.
-
No additional testing necessary.
APPROVED
classpath:
protocols in Spring
Ensure that SpringResourceAccessor uses classpath* protocol for loading resources if no other protocol is given.
Environment
Liquibase Version:
4.2.0
Liquibase Integration & Version: <Pick one: CLI, maven, gradle, spring boot, servlet, etc.>
maven
Liquibase Extension(s) & Version:
none
Database Vendor & Version:
none
Operating System Type & Version:
Windows 11
Pull Request Type
Description
A clear and concise description of the issue being addressed. Additional guidance here.
As described in #1540 it is not possible to provide changelogs from outside classpath. All locations will be searched on classpath only, even if a file protocol is provided. File protocol is used when database changelogs are handled outside the running applications as installation procedure is separated from deployment. So providing file protocol should be possible.
Steps To Reproduce
List the steps to reproduce the behavior.
Just specify changelog location with file protocol.
Actual Behavior
Location is changed to point to classpath.
Expected/Desired Behavior
When file protocol is given, this should remain unchanged.
Fast Track PR Acceptance Checklist:
Need Help?