blender/flamenco
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Don't return 500 Error when Blender cannot be found on $PATH

Browse Source 
In the first-time wizard, if Blender cannot be found on $PATH but it can
be found via .blend file association, that should just be reported as a
normal sitation, and not as a `500 Internal Server Error`.
This commit is contained in:
Sybren A. Stüvel 2022-07-14 18:50:34 +02:00
parent c0f4657be4
commit 62ecd09f5f
1 changed files with 1 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
internal/manager/api_impl/meta.go
View File

@ -184,8 +184,7 @@ func (f *Flamenco) FindBlenderExePath(e echo.Context) error {
case errors.Is(err, fs.ErrNotExist):
logger.Info().Msg("Blender could not be found as 'blender' on $PATH")
case err != nil:
logger.Warn().Err(err).Msg("there was an error finding Blender as 'blender' on $PATH")
return sendAPIError(e, http.StatusInternalServerError, "there was an error finding Blender: %v", err)
logger.Info().Err(err).Msg("there was an error finding Blender as 'blender' on $PATH")
default:
response = append(response, api.BlenderPathCheckResult{
IsUsable: true,