[wp-trac] [WordPress Trac] #45889: Include Session Tokens as personal information in data exports and erasure
WordPress Trac
noreply at wordpress.org
Wed Jan 29 10:27:05 UTC 2020
#45889: Include Session Tokens as personal information in data exports and erasure
-------------------------------------------------+-------------------------
Reporter: lakenh | Owner: garrett-
| eclipse
Type: enhancement | Status: accepted
Priority: normal | Milestone: 5.4
Component: Privacy | Version: 4.9.6
Severity: normal | Resolution:
Keywords: has-patch has-unit-tests has- | Focuses:
screenshots needs-testing | administration
-------------------------------------------------+-------------------------
Comment (by xkon):
Do we really need an extra group just for session data? We're adding a
filter (core values are protected) at #47509 to have an easier way to
include metadata that are simple & fast getters in the User meta section.
Since sessions are also within usermeta and they should be in the same
table IMHO.
On a further note since you can have multiple sessions per user, the table
view from [attachment:"45889.3.diff"] isn't optimal from a "reading"
perspective, see [attachment:"session_tokens.png"] as there's no clear
view of where 1 sessions data start and end.
I would prefer them under the existing "User meta data" (it's getting
renamed on the ticket mentioned earlier from `Users profile data` to
`Users meta data`) identified as Session Data group 1, Session data group
2 so on so forth, or with any other incremental name to again have an
easier identification while being read by someone. We can explode the
actual info with `<br/>` or something like that to separate the individual
data per session but essentially keep them within the same <tr> of the
table if that makes sense.
--
Ticket URL: <https://core.trac.wordpress.org/ticket/45889#comment:9>
WordPress Trac <https://core.trac.wordpress.org/>
WordPress publishing platform
More information about the wp-trac
mailing list