docker-archive / oscalkit

NIST OSCAL SDK and CLI
https://docker.github.io/oscalkit/
Other
36 stars 23 forks source link

Oscalkit failing to parse profile where an https import, imports a relative path. #70

Closed farhankamalkhan-10p closed 5 years ago

farhankamalkhan-10p commented 5 years ago

Related to: #57

When executing oscalkit generate code and processing a profile that has been imported using an https:// link, if that profile internally imports a profile/catalog relatively, the path is resolved incorrectly by appending the current system relative path to the https:// path of the parent profile that has imported the catalog/profile. The problem lies in the parentPath handing in: https://github.com/docker/oscalkit/blob/c5a92172971fa7ea1ccfdf7b08f67ed5bae53683/generator/profile.go#L44

cc: @minhaj10p @anweiss

anweiss commented 5 years ago

Good catch @farhan-khan30

minhaj10p commented 5 years ago

addressed in #71 @farhan-khan30 @anweiss

farhankamalkhan-10p commented 5 years ago

Verified fix in #71. Closing. @anweiss @minhaj10p