openapi: Add support for external refs #11076
Conversation
|
|
TheSpeedyFrog
force-pushed
the
iss8067_support_external_openai_ref
branch
from
645aa72 to
c3c773c
Compare
tpl/openapi/openapi3/openapi3.go
Outdated
| return nil, err | ||
| } | ||
|
|
||
| return ioutil.ReadAll(file) |
There was a problem hiding this comment.
| return ioutil.ReadAll(file) | |
| return io.ReadAll(file) |
ioutil is deprecated. See #10732
tpl/openapi/openapi3/openapi3.go
Outdated
| var relativePath string | ||
|
|
||
| if strings.HasPrefix(url.Path, "./") { | ||
| relativePath = strings.TrimRightFunc(key, func(r rune) bool { return r != '/' }) + strings.TrimPrefix(url.Path, "./") | ||
| } else { | ||
| relativePath = url.Path | ||
| } |
There was a problem hiding this comment.
| var relativePath string | |
| if strings.HasPrefix(url.Path, "./") { | |
| relativePath = strings.TrimRightFunc(key, func(r rune) bool { return r != '/' }) + strings.TrimPrefix(url.Path, "./") | |
| } else { | |
| relativePath = url.Path | |
| } | |
| relativePath := url.Path | |
| if strings.HasPrefix(url.Path, "./") { | |
| relativePath = strings.TrimRightFunc(key, func(r rune) bool { return r != '/' }) + strings.TrimPrefix(url.Path, "./") | |
| } |
| if err != nil { | ||
| return nil, err | ||
| } | ||
|
|
There was a problem hiding this comment.
Missing defer file.Close()
There was a problem hiding this comment.
just throwing this out there, but it looks like the contributor added the defer file.Close() on line 108
Set IsExternalRefsAllowed to true and provide a ReadFromURIFunc. Fixes gohugoio#8067
TheSpeedyFrog
force-pushed
the
iss8067_support_external_openai_ref
branch
from
c3c773c to
8ee09a6
Compare
|
This PR has been automatically marked as stale because it has not had recent activity. The resources of the Hugo team are limited, and so we are asking for your help. |
4 participants
Set IsExternalRefsAllowed to true and provide a ReadFromURIFunc.
Fixes #8067