mbr / simplekv

A simple key-value store for binary data.
http://simplekv.readthedocs.io
MIT License
152 stars 50 forks source link

Allow "/" in keys #31

Closed fmarczin closed 9 years ago

fmarczin commented 9 years ago

I was wondering what your opinion is on allowing / in keys.

The rationale behind it is that most if not all backends allow it, and some backends like S3 (AWS docs) and file systems handle this in a way that makes the keyspace much more organized.

I would see this purely as a change in some backends, and it's not supposed to affect the API semantics to make it in any way (pseudo-)hierarchical.

If you think this has merit, I would come up with a PR.

mbr commented 9 years ago

It's explicitly forbidden, to avoid confusion/security issues/clashes. / is basically the initial reason for having key names this restricted. Maybe simplekv will become hierarchical in the distant future, but I think that might even run afoul the "simple" part =).