Opened 22 months ago
Closed 15 months ago
#1488035 closed Bugs (fixed)
Database connection fails if database name contains an "@"
| Reported by: | fgp | Owned by: | |
|---|---|---|---|
| Priority: | 5 | Milestone: | 0.8-rc |
| Component: | Database | Version: | 0.5.3 |
| Severity: | normal | Keywords: | database at failure parseDSN |
| Cc: |
Description
If the database name contains an "@", roundcube is unable to connect to the database. This applies to both a literal "@" in the database DSN, as well as to an "@" represented by "%40". While the former case is not easily supportable, since there's no unambigouos way to parse a DSN which contains two "@" characters, ther latter case's failure seems like a bug.
The attached patch fixes that bug by making MDB2's parseDSN use rawurldecode() on the "database" field also, not only on "username" and "password" (where URL-escapes are handled correctly).
I've also reported this to the MDB2 guys, but since roundcube does include it's own copy of MDB2, I figured I'd report it here too. Here's a link to the upstream bugreport
Attachments (1)
Change History (2)
Changed 22 months ago by fgp
comment:1 Changed 15 months ago by alec
- Milestone changed from later to 0.8-rc
- Resolution set to fixed
- Status changed from new to closed
Patch applied in [4da69b95].
mdb2_parseDSN_database_rawurldecode