forked frompostgres/postgres
- Notifications
You must be signed in to change notification settings - Fork6
Commita7d2ada
Don't truncate database and user names in startup packets.
Unlike commands such as CREATE DATABASE, ProcessStartupPacket()does not perform multibyte-aware truncation of overlength names.This means that connection attempts might fail even if the userprovides the same overlength names that were used in CREATEDATABASE, CREATE ROLE, etc. Ideally, we'd do the same multibyte-aware truncation in both code paths, but it doesn't seem worth theadded complexity of trying to discover the encoding of the names.Instead, let's simply skip truncating the names in the startuppacket and let the user/database lookup fail later on. With thischange, users must provide the exact names stored in the catalogs,even if the names were truncated.This reverts commitd18c1d1.Author: Bertrand DrouvotReviewed-by: Kyotaro Horiguchi, Tom LaneDiscussion:https://postgr.es/m/07436793-1426-29b2-f924-db7422a05fb7%40gmail.com1 parentc2c1c4a commita7d2ada
1 file changed
+0
-9
lines changedLines changed: 0 additions & 9 deletions
Original file line number | Diff line number | Diff line change | |
---|---|---|---|
| |||
2290 | 2290 |
| |
2291 | 2291 |
| |
2292 | 2292 |
| |
2293 |
| - | |
2294 |
| - | |
2295 |
| - | |
2296 |
| - | |
2297 |
| - | |
2298 |
| - | |
2299 |
| - | |
2300 |
| - | |
2301 |
| - | |
2302 | 2293 |
| |
2303 | 2294 |
| |
2304 | 2295 |
| |
|
0 commit comments
Comments
(0)