Weird app names are allowed but throw errors later on

fly-apps / fly-laravel

Run your Laravel apps on Fly

42 stars 3 forks source link

Weird app names are allowed but throw errors later on #8

Closed Johannes-Werbrouck closed 1 year ago

Johannes-Werbrouck commented 1 year ago

If a user enters an app name like test ' ; 123, errors occur. We should add some checks or a regex to see if the appname is allowed before running fly launch with it.

KTanAug21 commented 1 year ago

PR https://github.com/fly-apps/fly-laravel/pull/23 fixes this, please close if so