it never pushes that error into @errors for me though (just gives a fatal error

With RaiseError => 1 you're telling DBI->connect() to raise an exception (i.e. die) in case of an error, rather than returning undef in combination with having set an error string.

So I think you should just get rid of it...