-
Notifications
You must be signed in to change notification settings - Fork 593
Closed
Labels
0.9.1Release v0.9.1Release v0.9.11.0.0Release v1.0.0Release v1.0.0bugSomething isn't workingSomething isn't working
Description
Version
main branch
Describe what's wrong
Fileset location is incorrect when getting path context from fileset with multiple locations,
Error message and/or stacktrace
See:
Lines 324 to 334 in 2271d58
} else { | |
// case 3: target location name is not passed in the header, and there are multiple locations. | |
// use the default location name | |
targetLocationName = fileset.properties().get(PROPERTY_DEFAULT_LOCATION_NAME); | |
// this should never happen, but just in case | |
Preconditions.checkArgument( | |
StringUtils.isNotBlank(targetLocationName), | |
"The default location name of the fileset %s should not be empty.", | |
fileset.name()); | |
targetLocation = fileset.properties().get(PROPERTY_DEFAULT_LOCATION_NAME); | |
} |
The targetLocation
here should not be obtained from fileset properties.
How to reproduce
Get credentials from fileset with multiple locations.
Additional context
No response
Metadata
Metadata
Assignees
Labels
0.9.1Release v0.9.1Release v0.9.11.0.0Release v1.0.0Release v1.0.0bugSomething isn't workingSomething isn't working