Installation - "The database hostname may not be greater than 50 characters." about crater

[leemcd56]

Update: Did a little looking around and found out the request validator in app/Requests/DatabaseEnvironmentRequest.php catches this. I don't believe it should have a max length because someone could connect to an external database.

In app\Space\EnvironmentManager.php there is a check on line 57 for database tables, but this command does not work in Postgres. When using \Schema::hasTable('users') instead (to check for preexisting migration) I was able to proceed with setup. It also successfully prevents setup if the database is already populated.

[mohitpanjwani]

Update: Did a little looking around and found out the request validator in app/Requests/DatabaseEnvironmentRequest.php catches this. I don't believe it should have a max length because someone could connect to an external database.

In app\Space\EnvironmentManager.php there is a check on line 57 for database tables, but this command does not work in Postgres. When using \Schema::hasTable('users') instead (to check for preexisting migration) I was able to proceed with setup. It also successfully prevents setup if the database is already populated.

Thanks for reporting the issue. We've removed the hostname validation of 50 characters to allow larger values.

Also, we've added a prefix to all the crater related tables and updated the DB Check to \Schema::hasTable('cr_users') so users can install crater in existing databases ( not forcing them to create a new DB specifically for crater).

Above changes will be added in the next update.