cartalyst/sentry — Issue #6: JOIN for metadata and IP in users table
Author: DregondRahl
Created: 2011-12-21 16:43:35+00:00
State: open

Just bringing these two points into consideration.

I think a JOIN on metadata saves a query. The reason it was separate was to stop conflicts in the column name i think, but most of the fields shouldn't have a duplicate in the metadata to begin with. The developer could just for example instead of 'email' in the metadata they could use 'contact_email' to stop any conflict.

I was also thinking maybe adding IP or login_ip to the users table, so when a user is registered/login their IP can be logged, if they are troublesome the IP is there to be blocked. Sure this can also be done in the metadata table but if 'login_ip' is to be used it can be updated with the 'login_hash'.

What are your thoughts on this?