[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