-
Notifications
You must be signed in to change notification settings - Fork 916
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
[BUG] importer.ReadManifest() attempts to read from relative path #3550
Comments
Howdy 🖐 Xiol ! Thank you for your interest in this project. We value your feedback and will respond soon. If you want to contribute to this project, please make yourself familiar with the |
Xiol
added a commit
to Xiol/govmomi
that referenced
this issue
Sep 18, 2024
Xiol
added a commit
to Xiol/govmomi
that referenced
this issue
Sep 18, 2024
11 tasks
Xiol
added a commit
to Xiol/govmomi
that referenced
this issue
Sep 18, 2024
Xiol
added a commit
to Xiol/govmomi
that referenced
this issue
Sep 18, 2024
Xiol
added a commit
to Xiol/govmomi
that referenced
this issue
Sep 20, 2024
Xiol
added a commit
to Xiol/govmomi
that referenced
this issue
Sep 23, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Describe the bug
I'm trying to use
importer.Import()
in my own application, with theVerifyManifest
option set totrue
.importer.ReadManifest()
appears to take the path to an .ovf file, gets the file from that path usingfilepath.Base()
, then changes the extension of that file to.mf
in order to find the manifest.However, at no point is the absolute path added back onto the resulting filename, so the reads will be done relative to the current working directory of the application.
To Reproduce
Steps to reproduce the behavior:
Import
with an absolute path to an OVF andVerifyManifest
set totrue
.Expected behavior
Manifest verification should succeed regardless of the current working directory of the application and where the manifest file is located.
Affected version
v0.43.0
Screenshots/Debug Output
N/A
Additional context
N/A
The text was updated successfully, but these errors were encountered: