<blockquote>
<p>Ideally I guess DBI->parse_dsn should be used but that would set a strong requirement on DBI 1.43+.</p>
</blockquote>
<p>We already depend on 1.57+, but there is another concern of loading DBI.pm as late as possible, thus I will stick with a regex.</p>
<blockquote>
<p>Do you need me to fix this PR with the DBI regex?</p>
</blockquote>
<p>No need, I will take it from here. Thank you!</p>
<p style="font-size:small;-webkit-text-size-adjust:none;color:#666;">—<br />You are receiving this because you are subscribed to this thread.<br />Reply to this email directly or <a href="https://github.com/dbsrgits/dbix-class/pull/99#issuecomment-204676743">view it on GitHub</a><img alt="" height="1" src="https://github.com/notifications/beacon/AASeAr14NnQ0vGgUktv0PI8CSwtdEEWxks5pzjHigaJpZM4H9qry.gif" width="1" /></p>
<div itemscope itemtype="http://schema.org/EmailMessage">
<div itemprop="action" itemscope itemtype="http://schema.org/ViewAction">
<link itemprop="url" href="https://github.com/dbsrgits/dbix-class/pull/99#issuecomment-204676743"></link>
<meta itemprop="name" content="View Pull Request"></meta>
</div>
<meta itemprop="description" content="View this Pull Request on GitHub"></meta>
</div>