Details
-
Improvement
-
Status: Done
-
Major
-
Resolution: Done
-
None
Description
This has hit quite a few people and could cause serious issues when upgrading from 6.x to 7. Be good to have a check for it.
DCP names are exposed when requesting the stats group dcp over the memcached protocol - we already have code that uses this data, should be possible to piggyback on it.
Attachments
Issue Links
- relates to
-
MB-34280 Memcached should disallow DCP connections with names greater than 200 characters (was: ns_server connections may be terminated if a client uses a long DCP_OPEN.name)
-
- Closed
-
Moving back to In Progress as I noticed a mistake in the patch (right after merging it...) - the tests have stream names in the format "eq_dcpq:replication:ns_1@127.0.0.1->ns_2@127.0.0.1:bucket:max_buffer_bytes" but they shouldn't include the stat name at the end (and since the tests pass, that implies the code itself is wrong).