openfaas-incubator/perl-template

Perl template for OpenFaaS

perl-template's Issues

Rename Handler.pm to avoid possible naming conflicts

Issue:
Handler.pm may exist elsewhere within one of standard module include paths. This may result in function not working due to different library search order than developer expects.

Proposed solution:
To avoid any confusion, rename Handler.pm to FaaSHandler.pm or similar.

cpanfile ignored during build process

Issue:
The empty cpanfile located in the template is always used during the build, ignoring any other cpanfile that user may create. This results in container with missing dependencies - not working at all unless there is no dependencies beyond core Perl module set.

Suggested fix/workaround:
Move cpanfile into function directory within template, copy directory first and use cpanminus rather than Carton to install dependencies from it.

Note:
Switching to cpanminus should also remove the need to use carton exec in fprocess and effectively remove one level of wrapper as library paths are added to main.pl anyway.

Lack of TLS support

Issue:
perl:<version>-slim image used here "only contains the minimal packages needed to run perl", meaning for example lack of TLS support, which is rather important in case of functions.

Proposed Fix:
This can be resolved in one of two ways:

1. Add required Debian modules to the Dockerfile and keep using 5.28.2-slim image (112MB)
2. Switch to full image 5.28.2 image (891MB)

For obvious reasons, first approach is preferred - it will increase the image size but not as much as switching to full version.