Merge pull request #4034 from fintura/master

Fix path variable being surrounded by quotes. Fixes #3464

changelog.md

@@ -2,6 +2,7 @@
 - [FEATURE] Geometry support for postgres
 - [BUG] Fix wrong count for `findAndCountAll` with required includes [#4016](https://github.com/sequelize/sequelize/pull/4016)
 - [BUG] Fix problems related to parsing of unique constraint errors [#4017](https://github.com/sequelize/sequelize/issues/4017) and [#4012](https://github.com/sequelize/sequelize/issues/4012)
+- [BUG] Fix postgres path variable being surrounded by quotes to often in unique constraint errors [#4034](https://github.com/sequelize/sequelize/pull/4034)
 
 # 3.3.2
 - [FIXED] upsert no longer updates with default values each time [#3994](https://github.com/sequelize/sequelize/pull/3994)

lib/dialects/postgres/query.js

@@ -371,7 +371,7 @@ Query.prototype.formatError = function (err) {
     case '23505':
       // there are multiple different formats of error messages for this error code
       // this regex should check at least two
-      match = errDetail.match(/Key \((.*?)\)=\((.*?)\)/);
+      match = errDetail.replace(/"/g, '').match(/Key \((.*?)\)=\((.*?)\)/);
 
       if (match) {
         fields = Utils._.zipObject(match[1].split(', '), match[2].split(', '));