question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Not able to customize ShouldRunChangeSetFilter.normalizePath in 4.x

See original GitHub issue

Issue: Till 3.10, ShouldRunChangeSetFilter used to allow overriding normalizePath (https://github.com/liquibase/liquibase/blob/3.10.x/liquibase-core/src/main/java/liquibase/changelog/filter/ShouldRunChangeSetFilter.java#L108), but it is not possible anymore in 4.x where this method has been moved to a static utility with no way of customization (https://github.com/liquibase/liquibase/blob/master/liquibase-core/src/main/java/liquibase/changelog/filter/ShouldRunChangeSetFilter.java#L102)

We have a use case where we don’t have a control over the source changelog files. We just need to import them to our system and run them. The problem here is that the path of the changelog files in both the systems are different, which causes all the changelogs to run everytime new changes are added to the imported files. Since we don’t have control over the source, we cannot set logicalFilePath. Hence we used the normalizePath in ShouldRunChangeSetFilter to remove the varying part of the path, which is not possible in 4.x version. We need to move to 4.x to fix a vulnerability CVE-2022-0839.

Can this be fixed or is there any other workaround?

Issue Analytics

  • State:open
  • Created a year ago
  • Comments:7 (4 by maintainers)

github_iconTop GitHub Comments

1reaction
tusharbhasmecommented, Jun 16, 2022

Liquibase Integration & Version: gradle:6.5.1

Liquibase Extension(s) & Version: liquibase-core:3.5.3

Database Vendor & Version: postgre:11

Operating System Type & Version: azul/zulu-openjdk-debian:8 (debian-bullseye-slim)

Infrastructure Type/Provider: AWS-EKS

0reactions
filipelautertcommented, Nov 1, 2022

Hi @santoshtrip - must not be related, this issue is about a code where they exetend a liquibase class. Which version of liquibase do you use?

Read more comments on GitHub >

github_iconTop Results From Across the Web

r - How to specify the path in normalizePath, or get around this ...
Rprofile file (just create it if it's not there yet) the following line: ... RStudio looks for R_USER first (and then moves on...
Read more >
normalizePath: Express File Paths in Canonical Form - Rdrr.io
An attempt is made not to introduce UNC paths in presence of mapped drives or symbolic links: if GetFinalPathNameByHandle returns a UNC path,...
Read more >
Path Class (System.IO) - Microsoft Learn
Path class members that modify a path string, such as ChangeExtension, have no effect on names of files in the file system. Some...
Read more >
Path | Node.js v19.2.0 Documentation
isAbsolute(path); path.join([...paths]); path.normalize(path) ... The node:path module provides utilities for working with file and directory paths.
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found