handle layers paths with trailing slash and leading ./ or just .

[dschep]

What did you implement:

handle layers paths with trailing slash
closes #5646

How did you implement it:

use path.resolve on the layer path before using it as a prefix

How can we verify it:

create a service with a layer with a path ending in /

Todos:

• Write tests
• Write documentation
• Fix linting errors
• Make sure code coverage hasn't dropped
• Provide verification config / commands / resources
• Enable "Allow edits from maintainers" for this PR
• Update the messages below

Is this ready for review?: YES
Is it a breaking change?: NO

[pmuens]

This looks good @dschep 🤘

Could you add some unit tests? After that it should be GTM !

[khalilgharbaoui]

There is more going on than just a bug with railing slashes check:
#5646 (comment)

[dschep]

@khalilgharbaoui, yup. I've noticed, just slow moving on this PR cause I've had other things on my plate. It's updated now and should handle that case too. Do you mind testing it? npm i -g serverless/serverless/#sls-5646

[dschep]

@pmuens, reworked the implementation and added tests for the new way of fixing this

[pmuens]

Thanks for updating @dschep 👍

Can conform that this works on my machine! LGTM

[khalilgharbaoui]

Working perfect now!